NCPP / ocgis

OpenClimateGIS is a set of geoprocessing and calculation tools for CF-compliant climate datasets.
Other
70 stars 19 forks source link

set mask to false for nc-region. #462

Closed huard closed 7 years ago

huard commented 7 years ago

Otherwise error is raised about missing mask when writing to disk. This is mysterious to me...

bekozi commented 7 years ago

Thanks, @huard. I wonder if this is a bug somewhere in the mask handling. How do I reproduce the "missing mask when writing to disk" error?

huard commented 7 years ago

Test file sent by email.

On Fri, Sep 8, 2017 at 10:02 AM Ben Koziol notifications@github.com wrote:

Thanks, @huard https://github.com/huard. I wonder if this is a bug somewhere in the mask handling. How do I reproduce the "missing mask when writing to disk" error?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NCPP/ocgis/pull/462#issuecomment-328111319, or mute the thread https://github.com/notifications/unsubscribe-auth/AAE9Q7loi8do4sxiFtJKgNHQzXDAyvxTks5sgUjcgaJpZM4PQd17 .

tomLandry commented 7 years ago

Great work guys!

huard commented 7 years ago

Good for me. Please go ahead.

On Fri, Sep 8, 2017 at 1:23 PM Tom Landry notifications@github.com wrote:

Great work guys!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NCPP/ocgis/pull/462#issuecomment-328164683, or mute the thread https://github.com/notifications/unsubscribe-auth/AAE9QzHC0Cy6DjRNrrnL7zcyW-Hsr2loks5sgXgsgaJpZM4PQd17 .

bekozi commented 7 years ago

Okay! Thanks for bringing up this disconnect. Keeping metadata and operations in sync is often challenging.

Also thanks @tomLandry for the props. Hope all is well!