-
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
Add Anomaly task to OTX 1.6.x #3906
Add Anomaly task to OTX 1.6.x #3906
Conversation
Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>
…extensions into ashwin/anomaly_geti_task_migration_1_6
…shwinvaidya17/training_extensions into ashwin/anomaly_geti_task_migration_1_6
Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>
…shwinvaidya17/training_extensions into ashwin/anomaly_geti_task_migration_1_6
Signed-off-by: Ashwin Vaidya <ashwinnitinvaidya@gmail.com>
Please do not merge this PR into releases/1.6.0 branch. This PR branch was just merged into aurelien/CVS-147615-Anomaly-reduction-documentation-update branch in Geti for early work on Geti side. Since this feature is targeting Geti 2.5, it should be merged in OTX2.2.x. |
@@ -43,6 +43,7 @@ | |||
"ACTION_CLASSIFICATION": "Custom_Action_Classification_X3D", | |||
"ACTION_DETECTION": "Custom_Action_Detection_X3D_FAST_RCNN", | |||
"VISUAL_PROMPTING": "Visual_Prompting_SAM_ViT_B", | |||
"ANOMALY": "otx_anomaly_padim", |
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.
this is defined here but never used anywhere. Do you think if this is needed? Or the other model template ids updated?
Same as #3910, the detection tests are failing here. Any thoughts @harimkang ? |
I'm not sure about 1.6. @chuneuny-emily Can you give us some advice? |
Summary
How to test
Checklist
License
Feel free to contact the maintainers if that's a concern.