Skip to content
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.

Sign up for GitHub

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

Jump to bottom

🔒 Bump onnx version as the previous version caused security issues #1907

Merged
merged 4 commits into from
Mar 26, 2024
Merged

🔒 Bump onnx version as the previous version caused security issues #1907

merged 4 commits into from
Mar 26, 2024

Conversation

samet-akcay
Copy link
Contributor

📝 Description

  • Bump onnx version as the previous version caused security issues

✨ 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.

AlexanderDokuchaev and others added 4 commits March 25, 2024 11:13
@AlexanderDokuchaev @samet-akcay
📚 Add action to check broken links (openvinotoolkit#1838)
7edb6af 
* Fix broken links

* Add link checker

* Fix link in template

* linter

* nightly check

---------

Co-authored-by: Samet Akcay <[email protected]>
@TurboJonte @samet-akcay
Use LabelName ENUM for label assignment (openvinotoolkit#1881)
8ebfdf5 
* Use LabelName ENUM for label assignment

* Update src/anomalib/deploy/inferencers/torch_inferencer.py

Co-authored-by: Samet Akcay <[email protected]>

---------

Co-authored-by: Samet Akcay <[email protected]>
@TurboJonte @samet-akcay
Changed prediction label assignment for OpenVINO inferencer (openvino…
6be4c80 
…toolkit#1872)

* Update openvino_inferencer.py

Fixed incorrect label assignment.
Was True/False, now it's "Anomalous/Normal" as for the PyTorch inferencer in torch_inferencer.py

* Fix ruff

---------

Co-authored-by: Samet <[email protected]>
@samet-akcay
Bump onnx version as the previous version caused security issues
7a775d9
@github-actions github-actions bot added Dependencies Pull requests that update a dependency file Setup labels Mar 26, 2024
@samet-akcay samet-akcay changed the base branch from main to release/1.0.1 March 26, 2024 07:47
@samet-akcay samet-akcay requested a review from paularamo as a code owner March 26, 2024 07:47
@samet-akcay samet-akcay merged commit bca285e into openvinotoolkit:release/1.0.1 Mar 26, 2024
8 checks passed
@samet-akcay samet-akcay deleted the security/bump-onnx branch March 26, 2024 08:23
samet-akcay added a commit that referenced this pull request Mar 27, 2024
@samet-akcay @djdameln
@AlexanderDokuchaev @TurboJonte @Bepitic @ashwinvaidya17
Merge v1.0.1 to main (#1905)
e35b574 
* Update CHANGELOG.md (#1899)

* Update __init__.py

* Fix metric bug (#1900)

* check metrics by name instead of key

* remove prefix from temp metric collection

* 🔒 Bump onnx version as the previous version caused security issues (#1907)

* 📚 Add action to check broken links (#1838)

* Fix broken links

* Add link checker

* Fix link in template

* linter

* nightly check

---------

Co-authored-by: Samet Akcay <[email protected]>

* Use LabelName ENUM for label assignment (#1881)

* Use LabelName ENUM for label assignment

* Update src/anomalib/deploy/inferencers/torch_inferencer.py

Co-authored-by: Samet Akcay <[email protected]>

---------

Co-authored-by: Samet Akcay <[email protected]>

* Changed prediction label assignment for OpenVINO inferencer (#1872)

* Update openvino_inferencer.py

Fixed incorrect label assignment.
Was True/False, now it's "Anomalous/Normal" as for the PyTorch inferencer in torch_inferencer.py

* Fix ruff

---------

Co-authored-by: Samet <[email protected]>

* Bump onnx version as the previous version caused security issues

---------

Co-authored-by: Alexander Dokuchaev <[email protected]>
Co-authored-by: TurboJonte <[email protected]>

* Add: fix image transform to float (#1902)

* 📚 Add action to check broken links (#1838)

* Fix broken links

* Add link checker

* Fix link in template

* linter

* nightly check

---------

Co-authored-by: Samet Akcay <[email protected]>

* Use LabelName ENUM for label assignment (#1881)

* Use LabelName ENUM for label assignment

* Update src/anomalib/deploy/inferencers/torch_inferencer.py

Co-authored-by: Samet Akcay <[email protected]>

---------

Co-authored-by: Samet Akcay <[email protected]>

* Changed prediction label assignment for OpenVINO inferencer (#1872)

* Update openvino_inferencer.py

Fixed incorrect label assignment.
Was True/False, now it's "Anomalous/Normal" as for the PyTorch inferencer in torch_inferencer.py

* Fix ruff

---------

Co-authored-by: Samet <[email protected]>

* Add: fix image transform to float

Signed-off-by: Bepitic <[email protected]>

* upd: video dtpe transform from torchvision

Signed-off-by: Bepitic <[email protected]>

* Added the scale.

Signed-off-by: Bepitic <[email protected]>

* Update src/anomalib/data/base/video.py

Co-authored-by: Dick Ameln <[email protected]>

---------

Signed-off-by: Bepitic <[email protected]>
Co-authored-by: Alexander Dokuchaev <[email protected]>
Co-authored-by: Samet Akcay <[email protected]>
Co-authored-by: TurboJonte <[email protected]>
Co-authored-by: Dick Ameln <[email protected]>

* Fix video bug and add tests (#1910)

* 📚 Add action to check broken links (#1838)

* Fix broken links

* Add link checker

* Fix link in template

* linter

* nightly check

---------

Co-authored-by: Samet Akcay <[email protected]>

* squeeze temporal dimension when clip length is 1

* add unit test for video input tensor dtype

* add test for single frame clips

* update changelog

* add author

---------

Co-authored-by: Alexander Dokuchaev <[email protected]>
Co-authored-by: Samet Akcay <[email protected]>

* Temporarily set devices to 1

* 🐞 Add greyscale conversion (#1911)

Add greyscale conversion

Signed-off-by: Ashwin Vaidya <[email protected]>

* 🐞 Fix publish workflow (#1912)

Disable tty prompt for gpg import

Signed-off-by: Ashwin Vaidya <[email protected]>

---------

Signed-off-by: Bepitic <[email protected]>
Signed-off-by: Ashwin Vaidya <[email protected]>
Co-authored-by: Dick Ameln <[email protected]>
Co-authored-by: Alexander Dokuchaev <[email protected]>
Co-authored-by: TurboJonte <[email protected]>
Co-authored-by: Paco <[email protected]>
Co-authored-by: Dick Ameln <[email protected]>
Co-authored-by: Ashwin Vaidya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ashwinvaidya17 ashwinvaidya17 ashwinvaidya17 approved these changes

@yunchu yunchu Awaiting requested review from yunchu

@djdameln djdameln Awaiting requested review from djdameln

@paularamo paularamo Awaiting requested review from paularamo

Assignees
No one assigned
Labels
Dependencies Pull requests that update a dependency file Setup
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@samet-akcay @ashwinvaidya17 @AlexanderDokuchaev @TurboJonte