New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ENH: Better handling of optional dependencies #49

Merged

ghisvail merged 1 commit into medkit-lib:main from ghisvail:ref/import-or-raise

May 17, 2024

Contributor

ghisvail commented May 14, 2024 •

edited

Loading

Inspired by Pandas design, tweaked for medkit's needs.

Importing a missing optional dependency raises a ModuleNotFound error with a note suggesting installing the appropriate medkit extra. Should avoid frustrating back and forth install of missing dependencies in an exploratory environment like Jupyter, where each kernel restart can become quite costly.
Package level symbols from optional modules will not be imported in case of missing dependencies.

ghisvail force-pushed the ref/import-or-raise branch 15 times, most recently from 5ca2f69 to ca61b19 Compare

May 17, 2024 13:14


          ENH: Improve handling of optional dependencies

ff67d1f

ghisvail force-pushed the ref/import-or-raise branch from ca61b19 to ff67d1f Compare

May 17, 2024 13:20

ghisvail marked this pull request as ready for review

May 17, 2024 14:15

ghisvail changed the title ~~REF: Better handling of optional dependencies~~ ENH: Better handling of optional dependencies

ghisvail merged commit 1869a9f into medkit-lib:main

8 checks passed

ghisvail deleted the ref/import-or-raise branch

May 17, 2024 14:18

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