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

update documentation FAQ about setting resources and turning off output #1608

Closed

Conversation

DeniseWorthen
Copy link
Collaborator

Description

Updates FAQ section of documentation to describe how to set resources for coupled model.

Top of commit queue on: TBD

Input data additions/changes

  • No changes are expected to input data.
  • There will be new input data.
  • Input data will be updated.

Anticipated changes to regression tests:

  • No changes are expected to any regression test.
  • Changes are expected to the following tests:

Subcomponents involved:

  • AQM
  • CDEPS
  • CICE
  • CMEPS
  • CMakeModules
  • FV3
  • GOCART
  • HYCOM
  • MOM6
  • NOAHMP
  • WW3
  • stochastic_physics
  • none

Combined with PR's (If Applicable):

Commit Queue Checklist:

  • Link PR's from all sub-components involved (N/A)
  • Confirm reviews completed in sub-component PR's
  • Add all appropriate labels to this PR.
  • Run full RT suite on either Hera/Cheyenne with both Intel/GNU compilers (N/A)
  • Add list of any failed regression tests to "Anticipated changes to regression tests" section. (N/A)

Linked PR's and Issues:

Test build can be seen here

Testing Day Checklist:

  • This PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR.
  • Move new/updated input data on RDHPCS Hera and propagate input data changes to all supported systems.

Testing Log (for CM's):

  • RDHPCS
    • Intel
      • Hera
      • Orion
      • Jet
      • Gaea
      • Cheyenne
    • GNU
      • Hera
      • Cheyenne
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
    • Completed
  • opnReqTest
    • N/A
    • Log attached to comment

@DeniseWorthen DeniseWorthen added the documentation Improvements or additions to documentation label Feb 10, 2023
@DeniseWorthen
Copy link
Collaborator Author

I just realized I skipped MOM6, so I will add that. I need to check w/ Jiande about what Jun's document had written.

* compute tasks = ``x layout * y layout * number of tiles``
* quilt tasks = ``write_groups * write_tasks_per_group`` if ``quilting==.true.``
In the UFS WM, each component's MPI task information, including the
starting task and end ending task numbers and the number of threads,
Copy link
Collaborator

Choose a reason for hiding this comment

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

'starting task and end ending task numbers and the number of threads,'

extra 'end'

and I think instead of 'task numbers' it's better to say 'task ranks'

FV3
^^^
FV3ATM consists of one or more forecast grid components and a write
grid component.
Copy link
Collaborator

Choose a reason for hiding this comment

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

write grid components


total tasks = number of forecast tasks + number of write grid component tasks

If ESMF-managed threading is used, the total number of tasks for
Copy link
Collaborator

@junwang-noaa junwang-noaa Feb 11, 2023

Choose a reason for hiding this comment

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

the total number of PETs for

::

ATM_petlist_bounds 0 total_tasks*number_of_threads-1
ATM_omp_num_threads number_of_threads
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe it will be clearer if we use number_of_threads_amt? I see we have "number_of_threads" for atm/wave/cmeps, we may let user know what the number_of_threads could be different for each component.

@DeniseWorthen
Copy link
Collaborator Author

@junwang-noaa @DusanJovic-NOAA I've made changes based on your feedback. I'm probably still mis-using the task/ranks terminology (I find the terms a bit squidgy). I'm happy to try to be more precise, but I'll need a little guidance if I haven't got it quite right.

Copy link
Collaborator

@junwang-noaa junwang-noaa left a comment

Choose a reason for hiding this comment

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

It looks good.

@jiandewang
Copy link
Collaborator

@DeniseWorthen : a question (may not related to this PR), under section
How do I select the file format for the model output (netCDF or NEMSIO)?
In your run directory, there is a file named model_configure. Change the variable output_file to 'netcdf' or 'nemsio'

are we still support nemsio ?

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Feb 13, 2023 via email

@jiandewang
Copy link
Collaborator

then we can take this chance to remove that in the doc

@DeniseWorthen DeniseWorthen changed the title update documentation FAQ about setting resources update documentation FAQ about setting resources and turning off output Feb 14, 2023
@jkbk2004
Copy link
Collaborator

@DeniseWorthen I will ask @binli2337 to combine in this pr. You can directly create a pr to his branch. Can you update the branch?

@DeniseWorthen
Copy link
Collaborator Author

@jkbk2004 I've updated but will wait for @binli2337 to update before making a PR.

@jkbk2004
Copy link
Collaborator

sure! I will leave a note to Bin's pr.

@DeniseWorthen DeniseWorthen changed the title update documentation FAQ about setting resources and turning off output update documentation FAQ about setting resources and turning off output; update FAQ documentation (was 1608); update drag suite intent mods (was 1612) Feb 21, 2023
@DeniseWorthen
Copy link
Collaborator Author

Combined with #1597; closing

@DeniseWorthen DeniseWorthen changed the title update documentation FAQ about setting resources and turning off output; update FAQ documentation (was 1608); update drag suite intent mods (was 1612) update documentation FAQ about setting resources and turning off output Feb 21, 2023
@DeniseWorthen DeniseWorthen self-assigned this Feb 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add information about setting resources for coupled configuration to the documentation
6 participants