-
Notifications
You must be signed in to change notification settings - Fork 36
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
SDAP-481: Enable internal compression for netcdf match up outputs #270
base: master
Are you sure you want to change the base?
Conversation
…o success until after all results are inserted
* pagination * update openapi spec * Fix for cdmslist bug in pagination PR (#263) * Fix for cdmslist bug * Fix for custom results type for cdmslist query --------- Co-authored-by: rileykk <[email protected]> * SDAP-473: Job priorization (#262) * Update changelog * job prioritization * fix typo * Add check for null __details * remove unnecessary print statement --------- Co-authored-by: Riley Kuttruff <[email protected]> Co-authored-by: rileykk <[email protected]>
Trying to verify, but the |
Hmm thanks @RKuttruff. I don't have mamba installed so my local build used Conda. I'll take a look and see if I can figure out which dependency is causing the mamba solve issue |
What I did to test this is that I commented out the analysis install and everything following it from the dockerfile then ran a shell in a container from that image so that I can run the mamba install myself to see what's going on. When trying to solve with the bumped dependency version, it fails after about 2+ hours with the message:
|
Also confirmed that the environment solves successfully without the |
I'm trying out different versions of Gdal now. thanks for bringing this to my attention |
SDAP-481
(NOTE: Will change PR target once 455 is merged)
netcdf4==1.6.4
. This is required as the compression params increateVariable
are only available in newer versions of this library.Tested this locally. Compared the output with the following command output:
Confirmed variables all contain
_DeflateLevel = 5
in the following output:Vardis also confirmed the outputs look as expected.
Idea for future improvement -- allow user to specify compression level? This is currently set to 5.