Skip to content

Handle missing netCDF4 dependency - #80

Draft
Mirochill wants to merge 1 commit into
castelao:mainfrom
Mirochill:fix-77-require-netcdf4
Draft

Handle missing netCDF4 dependency#80
Mirochill wants to merge 1 commit into
castelao:mainfrom
Mirochill:fix-77-require-netcdf4

Conversation

@Mirochill

Copy link
Copy Markdown

Summary

  • Keep a module-level netCDF4 = None marker when the optional NetCDF dependency is not installed.
  • Make cnv2nc() fail with an actionable ImportError that points to pip install 'seabird[CDF]' or pip install netCDF4 instead of continuing to a NameError.
  • Add a regression test for the missing-netCDF4 path.

Validation

  • git diff --check HEAD~1..HEAD
  • Static search for the new missing-dependency guard and regression test
  • Tests not run locally.

Fixes #77

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

'netCDF4' is not defined

1 participant