Skip to content

Conversation

@gustavocidornelas
Copy link
Contributor

…ed in the Jupyter notebook and the ones specified in the requirements file

Summary

Throws a warning if there are inconsistencies between the dependencies specified by in the requirements_txt_file and the modules installed in the notebook's env.

Sample warnings

If there is a dependency in requirements_txt_file that is not installed in the current env (in this case, the module pymupdf):
Screen Shot 2022-09-15 at 15 55 51
If there are version conflicts:
Screen Shot 2022-09-15 at 15 46 25

Testing

Churn notebook, running locally.

Copy link
Contributor

@whoseoyster whoseoyster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge after fixing the module level import issue

import pkg_resources
from bentoml.saved_bundle import bundler
from bentoml.utils import tempdir
from pkg_resources import DistributionNotFound, VersionConflict
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

follow google style guide on module level imports only

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed, thanks!

…ed in the Jupyter notebook and the ones specified in the requirements file
@gustavocidornelas gustavocidornelas merged commit 74ab2b2 into main Sep 30, 2022
@gustavocidornelas gustavocidornelas deleted the cid/env-validation branch September 30, 2022 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants