-
Notifications
You must be signed in to change notification settings - Fork 846
feat(engine): Support barebones Mode for Engine #3189
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
base: main
Are you sure you want to change the base?
feat(engine): Support barebones Mode for Engine #3189
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 implements barebones mode support for the Engine, which allows running training/testing with minimal overhead by disabling checkpointing and logging while still preserving metric collection.
Key changes:
- Barebones mode disables automatic checkpoint creation during training
- Manual metric population ensures metrics are available in barebones mode despite disabled logging
- Test coverage added to verify metric consistency and checkpoint behavior between normal and barebones modes
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/anomalib/engine/engine.py | Disables checkpoint callback creation when barebones mode is enabled and injects logged metrics into test results |
| src/anomalib/metrics/evaluator.py | Manually updates trainer metrics in barebones mode to ensure availability in return values |
| tests/unit/engine/test_engine.py | Adds comprehensive test verifying metric consistency and checkpoint behavior in barebones mode |
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
π‘ Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
π Description
Implements #3095
Information about barebones is in the issue.
β¨ Changes
Select what type of change your PR is:
β Checklist
Before you submit your pull request, please make sure you have completed the following steps:
For more information about code review checklists, see the Code Review Checklist.