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.

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

More documentation work: ush/enspost_config.py, ush/fire_emiss_tools.py, ush/interp_tools.py #520

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

edwardhartnett
Copy link
Contributor

DESCRIPTION OF CHANGES:

More documentation.

TESTS CONDUCTED:

N/A

Machines/Platforms:

N/A

Test cases:

N/A

ISSUE:

Part of #467

@edwardhartnett edwardhartnett changed the title More documentation work More documentation work: ush/enspost_config.py, ush/fire_emiss_tools.py, ush/interp_tools.py Oct 9, 2024
tgt_area: ???
beta: ???
fg_to_ug: ???
"""
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@haiqinli and @MatthewPyle-NOAA what do each of these parameters mean?

Copy link
Contributor

@MatthewPyle-NOAA MatthewPyle-NOAA Oct 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of these were already defined for HWP_tools.py - guessing these items mean the same here. Can @haiqinli confirm and fill out the rest?

         fcst_dates: a list of forecasts (for production/ops/ebb=2, it is {current_day - 25 hours: current_day-1hours}, for ebb=1, it is current_day:current_day+24 hours)
         cols: hard coded dimension for the RRFS_NA_3km and RRFS_CONUS_3km domains:: cols = 2700 if predef_grid == 'RRFS_NA_3km' else 1092
         rows: hard coded dimension for the RRFS_NA_3km and RRFS_CONUS_3km domains:: rows = 3950 if predef_grid == 'RRFS_NA_3km' else 1820
         intp_dir: the working directory for smoke processing (${CYCLE_DIR}/process_smoke)
         rave_to_intp: a string based on the grid name:: rave_to_intp = predef_grid+"intp"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JohanaRomeroAlvarez Johana is the author of these scripts, but she is on vacation this week and next week. @jordanschnell Jordan, would you like to help to check these parameters mean? Thanks.

current_day: ???
cols: ???
rows: ???
rave_to_intp: ???
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@haiqinli and @MatthewPyle-NOAA what do each of these parameters mean?


cols: ???
rows: ???
te: ???
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@haiqinli and @MatthewPyle-NOAA what do each of these parameters mean and what does this function do?

fire_age: ???
cols: ???
rows: ???
"""
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@haiqinli and @MatthewPyle-NOAA what do each of these parameters mean and what does this function do?

"""
???

"""
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@haiqinli and @MatthewPyle-NOAA what does this module hold?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants