Skip to content

Conversation

@abukowski21
Copy link
Collaborator

[FEAT/etc.] Add ability to postprocess to netCDF, clean up srcPython (some)

Some features, some house-keeping...

  • Postprocess to NetCDF
    • Only optional!
    • Does not change behavior without the netcdf-related flags. Does not require netCDF stuff to be installed w python
    • Can write 1 file for the entire run, or one file per timestep
    • lon/lat/z are dimensions (& variables when possible). Longitude/Latitude/Altitude are variables always (inter-compatibility)
    • works, without major changes, with PyITM
  • Cleaned up srcPython
    • python 2 code is in python2/ - this can probably be deleted?
    • Stuff that needs aetherpy is silo-ed too. Maybe not delete?
    • plot_drivers works with imf OR ae, and can take command line arguments
      • the plots without imf are especially ugly, but it works

Justification

Addresses #29 #54

Note

I'm open to debate on how the netcdf's are handled and whether we delete the silo-ed python codes.


Changes to GITM outputs

None.

Copy link
Contributor

@aaronjridley aaronjridley left a comment

Choose a reason for hiding this comment

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

These changes look fine to me. I think that we are moving the GITM and Aether output files closer toward each other, which is good.

I think that the GITM nc files should then work with pyitm, which is great. It is nice that we can now use the same plotting software for both. That is a huge step forward.

I am fine with moving the old plotting stuff into subdirectories.

@aaronjridley aaronjridley merged commit e410e22 into develop Sep 10, 2025
3 checks passed
@aaronjridley aaronjridley deleted the postprocnetcdf branch September 10, 2025 18:37
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.

3 participants