-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update ReadMe of ufs-coastal #47
Comments
A problem with modifying the readme in this repo is that it will constantly show a conflict when syncing with the upstream ufs-weathermodel fork. @pvelissariou1 and I will meet to discuss a solution. I'll look at ignoring readme when running git fetch or git pull |
This is old but might be a solution: https://stackoverflow.com/questions/25315687/ignore-specific-files-when-pulling-from-forked-upstream-origin/25315991#25315991 ...not sure if practical |
A draft README file (started from CoastalApp) is at the cmmb branch of UFS-Coastal (https://github.com/oceanmodeling/ufs-coastal/tree/cmmb). Also, there is a licence file that needs to be modified. We might use some material from the original README file (from ufs weather model). Will work on this branch before merging the changes to the main branch. |
Regarding UFS-Coastal and UFS weather model pulls, we need to establish a strategy on how to do this, and what to fetch from wather model. |
Sample introduction for ufs-coastal readme UFS-COASTAL open source code infrastructure forked from UFS-Weather-Model and its downstream applications (ufs-coastal-apps) is being developed as the next generation NOAA’s National Ocean Service (NOS) Coastal Ocean Modeling infrastructure. NOS is partnering with the Oceanic and Atmospheric Research (OAR), National Weather Service (NWS) and coastal ocean modeling community to develop UFS-Coastal to be integrated into the Unified Forecast System modeling portfolio. |
@saeed-moghimi-noaa |
ufs-coastal executable within a workflows to constitute an application(s):
|
@saeed-moghimi-noaa ok, thanks. I'll see how to incorporate these in a best way. This is good !!! |
notes from surge team tagup today:
|
I created a google doc shared with the team. |
I spoke to @saeed-moghimi-noaa yesterday and had some documentation questions. He referred me to this project. First, great job being so clear and transparent with the project goals and tasks! You're providing a wonderful example for us all! I'm glad the README is progressing and look forward to reviewing it. Hopefully my question is appropriate to ask here, if not, let me know and I'll create a new ticket. My question is about nomenclature and how we can be consistent throughout our docs and at all levels of the organization. Specifically, I'm struggling with the difference between Systems and Applications. I think the hierarchy that is emergins is something like:
I see two problems in our communications:
I think it's important to establish some common vocabulary as we're coordinating this enormous project and trying to communicate between developers, managers, external PIs and even NOAA leadership. Thanks for your thoughts. |
@dpsnowden Derick, thank you for your comments. Yes language is very important. Let me finalize the README draft and then we will iterate to finalize the document. The same goes for the license file for UFS-Coastal. |
@dpsnowden I'm not one to underestimate the power of communication, so I can definitely see this is important to get right in the ReadMe and elsewhere My understanding from the (relatively short) time I've been on this project, is that -- based on the nomenclature the UFS-Coastal team uses -- there are multiple "component" types, one of which is the model component:
The various coupling "applications" are also called "configurations" within ESMF documentation and elsewhere (including the way we talk about it internally). So maybe in that sense, if those two terms were better differentiated as coastal app, weather app, etc. & SCHISM+WW3 configuration, DATM+ROMS+CICE configuration, etc. ... then perhaps that helps solves the problem? @uturuncoglu @pvelissariou1 @saeed-moghimi-noaa please correct & apologies if I have misrepresented anything above |
Thanks, @janahaddad. It sounds like "applications" and "configurations" can be used interchangeably. I'll probably opt for "applications" in presentations because it feels a little more approachable and plain language for non-modelers. Since those seem distinct, I'll separate mediators and drivers in my figure (and presentations). I've been lumping them into a general "infrastructure code" category, but I don't see any harm in being specific. But this brings up another question that's separate from the vocabulary issue. What's the difference, or intended difference, between the ufs-coastal repo and the ufs-coastal-app repo? Those should be clear in the README's and in all the Sphinx docs too. I understand that there is some .gitignore wizardy that has to happen before we can make the repo's reflect the answer. But, in principle what's the difference? It doesn't seem to me that the ufs-coastal-app repo is an Application in the way we've been defining it here. On another note, the reason I'm digging into nomenclature so much is that I'm trying to document the full plan for:
I don't know if this repo is the right place (hopefully?) to document the roadmap but that's my hope. If I'm right, shouldn't "Transition to ESMX" be a phase/milestone/iteration on this roadmap? Last question, that I'll pose in another issue.... Is data assimilation part of the ufs-coastal? Or is JEDI sufficiently distinct that it should be addressed elsewhere? |
@dpsnowden At this stage data assimilation is not part of ufs-coastal. I have completed the firts draft of the README file that contains brief instructions for the compilation of UFS-Coastal. The file is currently shared with @saeed-moghimi-noaa , @janahaddad , @gseroka , @dpsnowden , @SorooshMani-NOAA , @BahramKhazaei-NOAA , @FariborzDaneshvar-NOAA , @[email protected], @uturuncoglu and @yunfangsun . Please take a look at the file and post (comments) any suggestions you might have. |
@dpsnowden , @saeed-moghimi-noaa , @janahaddad We might need to insert a disclaimer section at the end of the document. Also after the first round of reviews, we may start working on the wording of the document. |
04/03/2024 update
Finalize the draft by Monday 04/08 for further review |
@pvelissariou1 and the team, |
I'm done reviewing the document. Great work and thanks for the opportunity to weigh in. |
Closing this ticket and will open another to start moving readme, Wiki, etc. to coastal-app repo instead, per our recent discussions in the team and with @dpsnowden |
Description
There was request to include more information on the ReadMe
Solution
Bring some of the information from CoastalApp ReadMe.
Information like:
The text was updated successfully, but these errors were encountered: