Allow grouping of input files - #4
Conversation
|
Thanks @joshuatorrance for putting this together. The implementation with the Encodings I think we could drop the Inconsistent surface altitude This causes an issue with Not sure what the best way around this is, perhaps we could catch this error and use Dropping |
Inconsistent surface altitudeThis change should handle the inconsistent It adds an extra preprocessing step to just round off 'surface_altitude' to 4 decimal places. I've piggybacked this behaviour off of the |
Encodings
None of these are important for |
00343d8 to
6357169
Compare
blimlim
left a comment
There was a problem hiding this comment.
Still testing running it on full model output, but thought I'd put in my comments on the code in the meantime. I really like the implementation, and just had a couple of small comments
History now only build once Corrected repo URL
blimlim
left a comment
There was a problem hiding this comment.
Thanks @joshuatorrance, the outputs from this look good when testing it on a years worth of ESM1.6 data. I compared the results to um2nc's single variable output on yearly files produced natively by the model, and everything I checked was identical up to cosmetic differences.
Co-authored-by: Spencer Wong <88933912+blimlim@users.noreply.github.com>
…tcdf. Added option to skip existing files. Combined these options allow interrupted runs to recover faster, i.e. skip writing out to files that have already been finished
|
I just added to option to
Combined these two allow for a interrupted run to recover without having to re-write completed files. |

This PR adds a new feature to allow for the grouping of input files into output files. E.g. 12 monthly files into 1 yearly file per variable.
Closes #3
Closes #7