Skip to content

Conversation

@rajeshgangireddy
Copy link
Contributor

@rajeshgangireddy rajeshgangireddy commented Dec 4, 2025

πŸ“ Description

Implements #3095
Information about barebones is in the issue.

✨ Changes

Select what type of change your PR is:

  • πŸš€ New feature (non-breaking change which adds functionality)
  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • πŸ”„ Refactor (non-breaking change which refactors the code base)
  • ⚑ Performance improvements
  • 🎨 Style changes (code style/formatting)
  • πŸ§ͺ Tests (adding/modifying tests)
  • πŸ“š Documentation update
  • πŸ“¦ Build system changes
  • 🚧 CI/CD configuration
  • πŸ”§ Chore (general maintenance)
  • πŸ”’ Security update
  • πŸ’₯ Breaking change (fix or feature that would cause existing functionality to not work as expected)

βœ… Checklist

Before you submit your pull request, please make sure you have completed the following steps:

  • πŸ“š I have made the necessary updates to the documentation (if applicable).
  • πŸ§ͺ I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).
  • 🏷️ My PR title follows conventional commit format.

For more information about code review checklists, see the Code Review Checklist.

Copilot AI review requested due to automatic review settings December 4, 2025 11:29
@rajeshgangireddy rajeshgangireddy changed the title feat(engine) : Support barebones Mode for Engine feat(engine): Support barebones Mode for Engine Dec 4, 2025
Copy link
Contributor

Copilot AI left a 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.

Copilot AI review requested due to automatic review settings December 4, 2025 15:17
Copy link
Contributor

Copilot AI left a 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.

Copy link
Contributor

Copilot AI left a 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.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant