search

jgomezdans / KaFKA

Kascading Fast Kalman Assimilation
GNU General Public License v3.0
5 stars 8 forks source link

Netcdf group #11

Closed NPounder closed 7 years ago

NPounder commented 7 years ago

This makes 'group' an optional keyword argument when creating netcdf variables. This is necessary as gdal (at least version 2.1.0) can't read groups. (#10)

This update does break backwards compatibility as I've made group an optional keyword argument and moved its position in create_variable, create_empty_variable and update_variable to do so. If this is a problem then we can move it back and require the user to explicitly call group=None, but I'd prefer not to if we can cope with this change now.

still to-do. The time variable is not updated

NPounder commented 7 years ago

Added a method to update the time variable separately. This needs to be handled differently to other variables because adding a slice to one variable automatically expands the time dimension - so this can't be appended unless it is the first variable to be updated (which is another possiblility).