Skip to content

πŸ”’ Fix bandit and semgrep issues of Fuvas video anomaly detection model. #2655

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

Merged
merged 3 commits into from
Apr 10, 2025

Conversation

samet-akcay
Copy link
Contributor

πŸ“ Description

  • πŸ”’ Fix bandit and semgrep issues of Fuvas video anomaly detection model.

✨ Changes

Select what type of change your PR is:

  • 🐞 Bug fix (non-breaking change which fixes an issue)
  • πŸ”¨ Refactor (non-breaking change which refactors the code base)
  • πŸš€ New feature (non-breaking change which adds functionality)
  • πŸ’₯ Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • πŸ“š Documentation update
  • πŸ”’ Security update

βœ… Checklist

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

  • πŸ“‹ I have summarized my changes in the CHANGELOG and followed the guidelines for my type of change (skip for minor changes, documentation updates, and test enhancements).
  • πŸ“š 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).

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

Co-authored-by: Alexander Barabanov <[email protected]>
Signed-off-by: Samet Akcay <[email protected]>
@samet-akcay samet-akcay merged commit 67eb2f3 into open-edge-platform:feature/fuvas Apr 10, 2025
7 of 8 checks passed
@samet-akcay samet-akcay deleted the fix-fuvas branch April 10, 2025 08:16
samet-akcay added a commit that referenced this pull request Apr 10, 2025
* πŸš€ Add `FUVAS` Video Anomaly Detection Model (#2627)

* fuvas integrated

* fuvas integrated with changemd

* Update CHANGELOG.md

Signed-off-by: Samet Akcay <[email protected]>

* Rename fuvas to Fuvas

Signed-off-by: Samet Akcay <[email protected]>

* Address linting errors in feature extractor

Signed-off-by: Samet Akcay <[email protected]>

* Fix the linting errors in torch model

Signed-off-by: Samet Akcay <[email protected]>

* Update docstrings

Signed-off-by: Samet Akcay <[email protected]>

* use torchvision feature extractor

* fuvas readme and change np to torch

* fuvas readme modify

* update readme and python annotation

* add fvcore dependencies

* test model change

---------

Signed-off-by: Samet Akcay <[email protected]>
Co-authored-by: Jiaxiang Jiang <[email protected]>
Co-authored-by: Samet Akcay <[email protected]>

* πŸš€ Add FUVAS Video Anomaly Detection Model (#2652)

* Fix pre-commit

Signed-off-by: Samet Akcay <[email protected]>

* Skip video anomaly detection models for now

Signed-off-by: Samet Akcay <[email protected]>

---------

Signed-off-by: Samet Akcay <[email protected]>

* πŸ”’ Fix bandit and semgrep issues of Fuvas video anomaly detection model. (#2655)

* Address bandit and semgrep issues

Signed-off-by: Samet Akcay <[email protected]>

* Address bandit and semgrep issues

Signed-off-by: Samet Akcay <[email protected]>

* Update src/anomalib/models/__init__.py

Co-authored-by: Alexander Barabanov <[email protected]>
Signed-off-by: Samet Akcay <[email protected]>

---------

Signed-off-by: Samet Akcay <[email protected]>
Co-authored-by: Alexander Barabanov <[email protected]>

* πŸ—‘οΈ Remove `TaskType` from `FUVAS` Video Anomaly Detection Model (#2658)

remove task type from fuvas

Signed-off-by: Samet Akcay <[email protected]>

---------

Signed-off-by: Samet Akcay <[email protected]>
Co-authored-by: Jiaxiang Jiang <[email protected]>
Co-authored-by: Jiaxiang Jiang <[email protected]>
Co-authored-by: Alexander Barabanov <[email protected]>
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.

2 participants