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

Add netcdf debugging utilities #550

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SeanBryan51
Copy link
Collaborator

@SeanBryan51 SeanBryan51 commented Feb 23, 2025

This change adds a module for common NetCDF utilities and includes a to_netcdf subroutine to aid with debugging.

Type of change

  • New or updated documentation

Checklist

  • The new content is accessible and located in the appropriate section
  • I have checked that links are valid and point to the intended content
  • I have checked my code/text and corrected any misspellings

Testing

  • Are the changes bitwise-compatible with the main branch? If working on an optional feature, are the results bitwise-compatible when this feature is off? If yes, copy benchcab output showing successful completion of the bitwise compatibility tests or equivalent results below this line.

📚 Documentation preview 📚: https://cable--550.org.readthedocs.build/en/550/

Copy link
Collaborator Author

@SeanBryan51 SeanBryan51 Feb 23, 2025

Choose a reason for hiding this comment

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

  • Support for other types and and dimensions
  • Ford documentation
  • Code formatting
  • Testing

@SeanBryan51 SeanBryan51 force-pushed the netcdf-utils-for-debugging branch 10 times, most recently from 84177e8 to f142528 Compare February 24, 2025 01:38
@ccarouge
Copy link
Member

@SeanBryan51 In the API documentation, could we add some indication these subroutines are not to be used to add routine outputs from CABLE because the resulting netcdf files do not provide the required level of information. New outputs for the CABLE model should be implemented in cable_output.F90.
Just to prevent future confusion.

@SeanBryan51 SeanBryan51 force-pushed the netcdf-utils-for-debugging branch from f142528 to e97adcb Compare February 24, 2025 04:50
@SeanBryan51 SeanBryan51 force-pushed the netcdf-utils-for-debugging branch from e97adcb to 5846c9c Compare February 24, 2025 04:53
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.

2 participants