asascience-open / ncSOS

SOS Service on top of THREDDS
MIT License
15 stars 15 forks source link

Upgrade to netCDF-Java v5 #189

Open lesserwhirls opened 6 years ago

lesserwhirls commented 6 years ago

Greetings!

I recently took a stab at upgrading ncSOS to use the latest beta for netCDF-Java and the TDS (see my branch here). The idea is that the TDS v5 is getting "close to release"™️, and it'd be awesome if we could get ncSOS included with it, like we currently do with 4.6. Unfortunately, it became clear pretty quick that someone with more knowledge of ncSOS would need to take a crack at this :-(

It seems there are a lot of workarounds for deficiencies in the ucar.nc2.ft.point.* API in 4.6. I'm not super familiar with that part of the codebase, other than I know it's not super solid or feature complete, so I'm not too surprised. That said, the handling of point data has changed quite a bit in netCDF-java 5 - see these notes for details. I got to the point where it becamme difficult to understand the workaround, and if those were even needed with the new API in v5.

cheryldmorse commented 6 years ago

Thanks @lesserwhirls. We will take a look at upgrading ncSOS to the latest version of netCDF-Java