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

EXERCISE C notebook #85

Closed
wants to merge 2 commits into from
Closed

Conversation

NoraLoose
Copy link
Contributor

@NoraLoose NoraLoose commented Sep 16, 2024

In this exercise we make a blueprint from scratch, which consists of four steps:

  • Preparing the input data for a ROMS simulation (using ROMS-Tools)
  • Building a C-Star Case, which includes gathering
    • ROMS source model code (cstar.base.BaseModel and cstar.base.AdditionalCode)
    • ROMS input data from step 1 (cstar.base.InputDataset)
  • Compiling and running the ROMS simulation (using C-Star)
  • Making a C-Star blueprint so that steps 1, 2, and 3 are reproducible

The regridding step in ROMS-Tools is currently very inefficient - in particular, when being used with use_dask=False as required on perlmutter. Even though this is a small test case, it will take the user ~40 minutes to just create the input data. That is too slow. I may have to pick up CWorthy-ocean/roms-tools#43 again, and refactor the regridding so that we re-use the same weights, even though this is technically a milestone for the beta release.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@NoraLoose NoraLoose changed the title Add draft for EXERCISE C EXERCISE C notebook Sep 16, 2024
@NoraLoose NoraLoose marked this pull request as draft September 16, 2024 22:49
@NoraLoose
Copy link
Contributor Author

There is now a complete notebook draft for EXERCISE C. (I'm still waiting to hear back about the account_key we should use on perlmutter.)

@NoraLoose NoraLoose closed this Oct 16, 2024
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.

1 participant