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

return support for torch 1.8.2 (LTS) in PL 1.7.* #14086

Closed
paantya opened this issue Aug 8, 2022 · 14 comments
Closed

return support for torch 1.8.2 (LTS) in PL 1.7.* #14086

paantya opened this issue Aug 8, 2022 · 14 comments
Labels
feature Is an improvement or enhancement

Comments

@paantya
Copy link

paantya commented Aug 8, 2022

🐛 Bug

not support torch 1.8.2 (LTS) in 1.7 PL

Expected behavior

You can run training using torch 1.8.2 (or 1.8.1)

Additional context

this is still LTS and many enterprises use it for their sales and prod

Also that version 1.8.1-4 is the default in ubuntu 22 https://packages.ubuntu.com/search?keywords=libtorch&searchon=names&suite=jammy&section=all

cc @Borda @carmocca @akihironitta

@paantya paantya added the needs triage Waiting to be triaged by maintainers label Aug 8, 2022
@paantya
Copy link
Author

paantya commented Aug 8, 2022

@awaelchli awaelchli added feature Is an improvement or enhancement ci Continuous Integration and removed needs triage Waiting to be triaged by maintainers labels Aug 8, 2022
@awaelchli
Copy link
Contributor

@paantya Are you currently seeing errors being raised when using PyTorch 1.8 with Lightning 1.7?

@paantya
Copy link
Author

paantya commented Aug 8, 2022

@awaelchli

Problems were yesterday when we installed pl 1.7 -- torch 1.9 or greater

  • in slack scalazi that pl 1.7 works with torch version 1.9 and more (link above)

did not launch today, I can check tomorrow if any changes were made)

@paantya
Copy link
Author

paantya commented Aug 9, 2022

@awaelchli

The dependencies seem to indicate torch >=1.9.*

image

@akihironitta
Copy link
Contributor

#13155 dropped support for PyTorch 1.8 (LTS) because (I remember) we'd like to keep the window size of four PyTorch versions (currently 1.9-1.12).

However, in addition to this issue, there's another user asking for PyTorch 1.8 (LTS) support (#14094 (comment)) after PL 1.7 release.

@carmocca @awaelchli @Borda @rohitgr7 If PyTorch 1.8 ends its life as LTS very soon (for example, within three months until the next feature release), I don't think it's worth adding it back, but if 1.8 will live as LTS for more than a few months, I believe it is. Do you think PL should support PyTorch 1.8 again?

@akihironitta
Copy link
Contributor

The dependencies seem to indicate torch >=1.9.*

@paantya Yes, but you can force-install PyTorch 1.8 after installing PL:

$ pip install pytorch-lightning -U
$ pip install torch==1.8.2 --extra-index-url https://download.pytorch.org/whl/lts/1.8/cpu  # for CPU

@paantya
Copy link
Author

paantya commented Aug 9, 2022

after install:

Installing collected packages: torch, pytorch-lightning
  Attempting uninstall: torch
    Found existing installation: torch 1.8.2+cu111
    Uninstalling torch-1.8.2+cu111:
      Successfully uninstalled torch-1.8.2+cu111
  Attempting uninstall: pytorch-lightning
    Found existing installation: pytorch-lightning 1.5.9
    Uninstalling pytorch-lightning-1.5.9:
      Successfully uninstalled pytorch-lightning-1.5.9
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
torchvision 0.9.2+cu111 requires torch==1.8.2, but you have torch 1.12.1 which is incompatible.
torchaudio 0.8.2 requires torch==1.8.2, but you have torch 1.12.1 which is incompatible.
Successfully installed pytorch-lightning-1.7.0 torch-1.12.1

@akihironitta Interesting crutch, thanks I'll try

this makes sense, do you happen to know how long torch will support 1.8.2?

@akihironitta
Copy link
Contributor

do you happen to know how long torch will support 1.8.2?

cc @carmocca

@carmocca
Copy link
Contributor

carmocca commented Aug 9, 2022

I asked if there's an updated LTS schedule for PyTorch in their developer Slack and got no answer. I would respect our support window. If there's enterprise interest in an LTS PL version, then they should contact us to discuss it. A solution could be to include critical bug fixes into the 1.6.x branch, but not support PyTorch 1.8 endlessly across versions.

@paantya
Copy link
Author

paantya commented Aug 9, 2022

@carmocca
Copy link
Contributor

carmocca commented Aug 9, 2022

@paantya If you have access to the PyTorch slack, this was also asked in https://pytorch.slack.com/archives/C3PDTEV8E/p1651758244430409 and https://pytorch.slack.com/archives/C3PDTEV8E/p1657621331393819

I also want to clarify that we chose to drop support for torch 1.8 after seeing the lack of a clear PyTorch LTS roadmap

@paantya
Copy link
Author

paantya commented Aug 9, 2022

@carmocca
Unfortunately there is no access, thanks for keeping up to date, it's cool!

there nothing was written about new versions of LTS or something like that?
or the guys just do not answer this question or do not know themselves?

@paantya
Copy link
Author

paantya commented Aug 10, 2022

while there is a suspended state with the version, I will request what we need #14142

@akihironitta
Copy link
Contributor

akihironitta commented Aug 11, 2022

Closing this issue as there seems no actionable item regarding PL's re-support for PyTorch 1.8. Let's discuss any particular feature requests in corresponding issues, such as #14142.

@akihironitta akihironitta closed this as not planned Won't fix, can't repro, duplicate, stale Aug 11, 2022
@akihironitta akihironitta removed the ci Continuous Integration label Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Is an improvement or enhancement
Projects
None yet
Development

No branches or pull requests

4 participants