search

Reading-eScience-Centre / edal-java

Environmental Data Abstraction Layer libraries
Other
39 stars 30 forks source link

Upgrade the Unidata library #168

Closed allanfrankwork closed 2 months ago

allanfrankwork commented 2 months ago

Hi

Simple PR for upgrading the Unidata libs to a newer version. I see similar work (and much more) is already done here: https://github.com/Reading-eScience-Centre/edal-java/compare/master...Unidata:edal-java:unidata_main.

I had to do some quick hack (not checked in) to get the h2 database to work locally on Windows and built a -SNAPSHOT version that I could test our service with. Maven built without errors and our server functioned without issue.

Reason for upgrade was Netcdf files and that has been refactored in the current version leaving only deprecated methods for creating netcdf files.

Thanks for a great library.

guygriffiths commented 2 months ago

Great, thanks for this. Sorry for the hassle, but can you please re-add it, but merging to develop instead of master?

allanfrankwork commented 2 months ago

Sure, I can do that. Opened https://github.com/Reading-eScience-Centre/edal-java/pull/169