-
Notifications
You must be signed in to change notification settings - Fork 343
Update documentation for generating fsurdat/landuse files #3199
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
base: b4b-dev
Are you sure you want to change the base?
Conversation
|
@slevis-lmwg note to self: Reminder to also see my updates from a while ago (2023/12/5) to |
|
@slevis-lmwg Marking this as draft since it seems like it's still in progress. Please undo that if I'm wrong. |
|
Since I may not look at this for a little while... |
I agree with @samsrabin I think we should bring in the README file in the embeded way, so that we only have to make changes in one place. This will be easier for maintence. The user's guide could have something additional, and maybe some figures that aren't in the README, but should be supplementary to the README. |
ff0c3ca to
42098c5
Compare
ekluzek
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should pare this down, so it's mostly just the include of the README.md file. That way we just update most of the instructions in one place rather than both. The parts at the top can be the more abstract kind of things that aren't likely to change. So probably just some of the introductory parts at the top would be kept.
We should also think about if we need any figures to illustrate any of the process. I think that was important for mksurfdata_esmf, but may not be as important here. Although I also think some of those prior figures could be adapted to illustrate how mksurfdata_esmf works.
Description of changes
I started with the README.md already available in /mksurfdata_esmf to update the relevant documentation file.
Specific notes
Contributors other than yourself, if any:
CTSM Issues Fixed (include github issue #):
Resolves #1718
Are answers expected to change (and if so in what way)?
No
Does this create a need to change or add documentation? Did you do so?
Yes
Testing performed, if any:
The build worked on my Windows laptop, but the browser would not open for me to evaluate the outcome. I was working in an .\slevisadmin space on Ubuntu, so next I should make a new clone in the non-admin Ubuntu space when I get back to updating and testing this work.