-
Notifications
You must be signed in to change notification settings - Fork 446
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
Fix the Anomaly Task to work & Add integration test for Anomaly #3007
Fix the Anomaly Task to work & Add integration test for Anomaly #3007
Conversation
…nsions into harimkan/anomaly-intg-test
…nsions into harimkan/anomaly-intg-test
…nsions into harimkan/anomaly-intg-test
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.
Looks good to me.
Let's see how we can sort out TorchMetrics PrecisionRecallCurve
issue.
…nsions into harimkan/anomaly-intg-test
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## v2 #3007 +/- ##
==========================================
- Coverage 61.96% 61.93% -0.03%
==========================================
Files 259 259
Lines 15167 15174 +7
==========================================
Hits 9398 9398
- Misses 5769 5776 +7
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@harimkang I checked out your branch and added the changes from #3057. Except for the OpenVINO tests, the rest of the integration tests are passing. Maybe you can merge the changes from that PR to yours |
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.
Thanks for all the efforts!
OV-related features will follow in this PR (#3057), and tests will be added in that PR. |
Summary
Modify the Anomaly Task so that the integration Test works.
Add a dependency of anomalib[core] for Anomalib to work properly.
This PR will be opened once
anomalib1.0
is working withtorchmetrics==1.3.1
.For now, install based on the hot-fix commit on the
anomalib
side.Add hazelnut dataset for anomaly integration test
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.