-
Notifications
You must be signed in to change notification settings - Fork 284
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.
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
feature: [VRD-682] Client utility function to translate import module names into distribution package names #3652
Conversation
…dule name are correctly captured
…arty package names
…a given set of import module names
… dependency checking
…ncluded packages is included in the env and return a more useful error message for missing packages
client/verta/tests/unit_tests/registry/test_check_model_dependencies.py
Outdated
Show resolved
Hide resolved
client/verta/tests/unit_tests/registry/test_model_dependencies.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Liu <96442646+liuverta@users.noreply.github.com>
client/verta/tests/unit_tests/registry/test_check_model_dependencies.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I re-opened a prior comment thread: #3652 (comment). We should have some test coverage of multiple missing modules.
That should be the last thing! Everything else looks good.
Co-authored-by: Liu <96442646+liuverta@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Impact and Context
This is the missing piece of functionality for the client method
check_model_dependencies
to work properly.The importlib.metadata module is used to capture info about the distribution packages for each of the import modules extracted from a model class.
Risks and Area of Effect
Fairly low risk, as this is net new functionality on the first iteration and it's all hidden from the docs for now.
The Python ecosystem does not enforce package naming conventions, and the
importlib
docs warn:This seems to be related specifically to editable installs. However I experimented with
verta
as aneditable
installation, and thename
attribute we need is there.Testing
Reverting