-
Notifications
You must be signed in to change notification settings - Fork 709
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
Added cflow algorithm #47
Conversation
@blakshma pipeline fails because of missing |
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.
Minor comment but rest looks good.
Thank you for updating requirements @samet-akcay |
@samet-akcay the pipeline fails while testing load model for cflow. Is this something you have encountered before? Any leads? |
@blakshma , you could look at the details here, which shows why it failed. It's mainly becuase of a CUDA memory issue while running C-Flow. |
…ib into algo/barath/cflow
…o algo/barath/cflow
It's quite likely that this is due to hardware differences between your machine and the mlops machine on which we run the CI. We'll investigate whether evaluation the metrics on cpu would remedy the issue. |
@@ -0,0 +1,3 @@ | |||
# Real-Time Unsupervised Anomaly Detection via Conditional Normalizing Flows | |||
|
|||
This is the implementation of the [CW-AD](https://arxiv.org/pdf/2107.12571v1.pdf) paper. |
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.
Do you have sample results? It would be nice to include them here for consistency with other tasks. Here is an example https://github.com/openvinotoolkit/anomalib/blob/development/anomalib/models/stfpm/README.md
Otherwise, can you create a ticket for it to address this in a different PR.
…olkit/anomalib into algo/barath/cflow
No description provided.