Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Parameters (in code) need comments #437

Closed
rgknox opened this issue Nov 2, 2018 · 0 comments · Fixed by #1009
Closed

Parameters (in code) need comments #437

rgknox opened this issue Nov 2, 2018 · 0 comments · Fixed by #1009
Assignees
@rgknox
Labels
clean up Simple issues like typos, formatting, style guide alignment, etc. enhancement

Comments

@rgknox
Copy link
Contributor

rgknox commented Nov 2, 2018

The parameters defined in EDPFtvarcon.F90 and EdParamsMod.F90 need comments for each variable describing verbosely what these parameters are and what their units are.
@rgknox rgknox mentioned this issue Nov 25, 2018
Merged
2 tasks
@glemieux glemieux moved this to ❕Todo in FATES issue board May 19, 2022
@rgknox rgknox added clean up Simple issues like typos, formatting, style guide alignment, etc. enhancement and removed enhancement clean up Simple issues like typos, formatting, style guide alignment, etc. labels Sep 1, 2022
@glemieux glemieux mentioned this issue Apr 3, 2023
Merged
5 tasks
@github-project-automation github-project-automation bot moved this from ❕Todo to ✔ Done in FATES issue board Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rgknox rgknox

Labels
clean up Simple issues like typos, formatting, style guide alignment, etc. enhancement
Projects
FATES issue board
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Split associate statement to avoid NAG compiler line continuation issue glemieux/fates
1 participant
@rgknox