-
Notifications
You must be signed in to change notification settings - Fork 22
CLI fixes #465
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
Conversation
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.
Pull Request Overview
This PR updates the CLI summary handling and version reporting, enhances test invocation for the new flags, and adds a VS Code environment configuration.
- import the
bioimageio.spec
module to use__version__
- change
WithSummaryLogging.summary
to always be a sequence and update its default - loop over two summary flag names in the resource tests and stop after the first successful summary
- add a
python-envs.pythonProjects
block to.vscode/settings.json
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
File | Description |
---|---|
bioimageio/core/cli.py | Imported bioimageio.spec , adjusted summary field type and version print |
bioimageio/core/_resource_tests.py | Added a loop over summary flag names and break logic in test invocation |
.vscode/settings.json | Introduced project-specific Python environment settings |
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified Files
|
No description provided.