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

Automated post-processing #3

Open
aidanheerdegen opened this issue Apr 29, 2024 · 8 comments
Open

Automated post-processing #3

aidanheerdegen opened this issue Apr 29, 2024 · 8 comments
Labels
priority:blocker type:release Required for next release

Comments

@aidanheerdegen
Copy link
Member

The UM outputs are in a proprietary format (PP files) that should be automatically post-processed to netCDF.

This has traditionally been done with python scripts like um2netcdf.py.

This functionality (amongst others) has been subsumed into a new package:

https://github.com/ACCESS-NRI/amami

which will provide a um2nc tool.

This will require having this package available in the payu conda-environment and adding a payu userscript hook to model configurations to call the um2nc function at the appropriate time.

@aidanheerdegen aidanheerdegen added priority:blocker type:release Required for next release labels Apr 29, 2024
@truth-quark
Copy link

truth-quark commented Apr 29, 2024

Do we have any indication if um2nc will be run from the command line/scripts and/or called as a library from workflow code?

@aidanheerdegen
Copy link
Member Author

Do we have any indication if um2nc will be run from the command line/scripts and/or called as a library from workflow code?

Likely called as a command line script at least in the first instance.

@aidanheerdegen
Copy link
Member Author

What are the requirements for post-processing? Review of command line options. What options can we exclude?

Is everything that is currently done necessary:

  • Coordinate rounding
  • Removing unreliable cell methods
  • Metadata fixes (coordinate naming,
  • Masking pressure level field
  • Fixing pressure coordinates
  • Converting to 32 bit
  • Calendar fixes

Any additional metadata required to meet FAIR data standards?

Pinging @MartinDix for feedback.

@atteggiani
Copy link

What should be defined here are all the requirements for the ACCESS-ESM1.5 post-processing conversion to netCDF.

Most importantly, I need to understand if there are any requirements that currently cannot be performed in amami um2nc, so I could work on the implementation of them.

Some examples could be:

  • Do we expect any processed um fieldsfile to contain time-series fields?

  • Do we expect the STASH variables lookup for the naming of the converted netCDF variables to use the same STASHmaster file or we need to allow the choice of a different STASHmaster files to be used for it?

@truth-quark
Copy link

It would also be useful to review other details such as any redundant um2nc edge cases. Ideally, anything that should be removed would be done "now", before retrofitting amami unit tests.

@aidanheerdegen
Copy link
Member Author

aidanheerdegen commented Jun 5, 2024

@MartinDix can you select which of these are required for the ESM1.5 post-processing?

  • Coordinate rounding
  • Removing unreliable cell methods
  • Metadata fixes (coordinate naming)
  • Masking pressure level field
  • Fixing pressure coordinates
  • Converting to 32 bit
  • Calendar fixes (proleptic Gregorian conversion)
  • Support time-series
  • Support choice of STASHmaster file

@truth-quark
Copy link

@MartinDix can you select which of these are required for the ESM1.5 post-processing?

  • Coordinate rounding
  • Removing unreliable cell methods
  • Metadata fixes (coordinate naming)
  • Masking pressure level field
  • Fixing pressure coordinates
  • Converting to 32 bit
  • Calendar fixes (proleptic Gregorian conversion)
  • Support time-series
  • Support choice of STASHmaster file

These are raised as issues/requirements here:
https://github.com/ACCESS-NRI/um2nc-standalone/issues

@truth-quark
Copy link

Removing unreliable cell methods is handled here: ACCESS-NRI/um2nc-standalone#73

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:blocker type:release Required for next release
Projects
None yet
Development

No branches or pull requests

3 participants