Skip to content

python tool that converts a netcdf parameter file to json#1557

Open
rgknox wants to merge 3 commits into
NGEET:mainfrom
rgknox:netcdf_to_json
Open

python tool that converts a netcdf parameter file to json#1557
rgknox wants to merge 3 commits into
NGEET:mainfrom
rgknox:netcdf_to_json

Conversation

@rgknox
Copy link
Copy Markdown
Contributor

@rgknox rgknox commented Apr 14, 2026

Description:

As the title says, this tool converts a netcdf binary to JSON. It uses xarray as an intermediary.

Collaborators:

@adrifoster

Expectation of Answer Changes:

Description of generative AI usage (as necessary)

Yes, this used generative AI to create the tool from the ground up through conversational design. The tool was then modified to use our write_json script. The logic in the code is not complicated and was reviewed by the author and makes sense. The tool was then tested on existing netcdf binaries.

Checklist

If this is your first time contributing, please read the CONTRIBUTING document.

All checklist items must be checked to enable merging this pull request:

Contributor

  • The in-code documentation has been updated with descriptive comments
  • The documentation has been assessed to determine if updates are necessary
  • Describe use of generative AI (if necessary)

Integrator

  • FATES PASS/FAIL regression tests were run N/A
  • Evaluation of test results for answer changes was performed and results provided N/A
  • FATES-CLM6 Code Freeze: satellite phenology regression tests are b4b N/A

If satellite phenology regressions are not b4b, please hold merge and notify the FATES development team.

Documentation

Test Results:

CTSM (or) E3SM (specify which) test hash-tag:

CTSM (or) E3SM (specify which) baseline hash-tag:

FATES baseline hash-tag:

Test Output:

Copy link
Copy Markdown
Contributor

@glemieux glemieux left a comment

Choose a reason for hiding this comment

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

Thanks for getting this tool going. I have a couple suggested changes and two questions.

Comment thread tools/netcdf_to_json.py Outdated
Comment thread tools/netcdf_to_json.py
Comment thread tools/netcdf_to_json.py Outdated
Comment thread tools/netcdf_to_json.py Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Under Review

Development

Successfully merging this pull request may close these issues.

2 participants