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.

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] Prevent torch 2.0.0 installation #1896

Merged
merged 3 commits into from
Mar 17, 2023
Merged

[FIX] Prevent torch 2.0.0 installation #1896

merged 3 commits into from
Mar 17, 2023

Conversation

JihwanEom
Copy link
Contributor

Summary

  • Add torchvision & torchtext in requirements/anomaly.txt with fixed version (0.14.1) to prevent torch 2.0.0 installation

How to test

With cuda 11.7 environment:

pip install torch==1.13.1 torchvision==0.14.1 --extra-index-url https://download.pytorch.org/whl/cu117
pip install -e .[full]

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2023 Intel Corporation
#
# SPDX-License-Identifier: MIT

@JihwanEom JihwanEom requested a review from a team as a code owner March 16, 2023 15:24
@JihwanEom JihwanEom added this to the 1.1.0 milestone Mar 16, 2023
@github-actions github-actions bot added the DEPENDENCY Any changes in any dependencies (new dep or its version) should be produced via Change Request on PM label Mar 16, 2023
@JihwanEom
Copy link
Contributor Author

CI failed by codecov coverage upload issue, but passed all of unit tests. (known issue)

Copy link
Contributor

@goodsong81 goodsong81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! Thank you for the quick fix.

@goodsong81 goodsong81 merged commit 8f7b3b7 into openvinotoolkit:develop Mar 17, 2023
goodsong81 pushed a commit that referenced this pull request Mar 17, 2023
* Add torchvision & torchtext in requirements/anomaly.txt with fixed version

* Update requirements/anomaly.txt

* Update requirements/anomaly.txt
goodsong81 added a commit that referenced this pull request Mar 17, 2023
* [RELEASE][DOC] Fix wrong info in documentation (#1849)
* updated dataset formats info. Fix for multilabel classification
* revert file
* revert file. minor
* added warning to instance segmentation
* revert changes

* [Enhance] Separate installation for each tasks on release task (#1869)
* separte_import
* align with pre commit
* update unit test code
* add separate task env pre  merge test & aplly it to github action
* add multiprocess to requirement

* [FIX][REL1.0] Fix Geti integration issues (#1885)
* Fix ote_config -> otx_config

* [FIX] hang issue when tracing a stack in certain scenario (#1868)
fix: use primitive library

* [FIX][POT] Set stat_requests_number parameter to 1 (#1870)
Set POT stat_requests_number parameter to 1 in order to lower RAM footprint

* [FIX] Training error when batch size is 1 (#1872)
fix: drop last batch

* Recover detection num_workers=2
* Remove nbmake from base requirements
* Add py.typed in package

* [FIX] Arrange scale between bbox preds and bbox targets in ATSS (#1880)
Arrange scale between bbox preds and bbox targets

* [FIX][RELEASE1.0] Remove cfg dump in ckpt (#1895)
* Remove cfg dump in ckpt
* Fix pre-commit

* Release v1.0.1

* [FIX] Prevent torch 2.0.0 installation (#1896)

* Add torchvision & torchtext in requirements/anomaly.txt with fixed version

* Update requirements/anomaly.txt

* Fix _model_cfg -> _recipe_cfg due to cfg merge

---------

Signed-off-by: Songki Choi <songki.choi@intel.com>
Co-authored-by: Prokofiev Kirill <kirill.prokofiev@intel.com>
Co-authored-by: Eunwoo Shin <eunwoo.shin@intel.com>
Co-authored-by: Inhyuk Cho <andy.inhyuk.jo@intel.com>
Co-authored-by: Nikita Savelyev <nikita.savelyev@intel.com>
Co-authored-by: Jaeguk Hyun <jaeguk.hyun@intel.com>
Co-authored-by: Jihwan Eom <jihwan.eom@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DEPENDENCY Any changes in any dependencies (new dep or its version) should be produced via Change Request on PM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants