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

PyTorch Lightining V1.0 requires fsspec>0.8 which clashes with Kedro 0.16.5 #553

Closed
ssakhavi opened this issue Oct 9, 2020 · 9 comments
Closed
Labels
Issue: Bug Report 🐞 Bug that needs to be fixed

Comments

@ssakhavi
Copy link

ssakhavi commented Oct 9, 2020

Description

The upcoming version of PyTorch Lightning requires fsspec V0.8>0, but Kedro has a dependency V0.7<

Context

My data science pipeline is dependent on PyTorch-Lightning and of I were to update it, it will clash with Kedro

  • Kedro version used (pip show kedro or kedro -V): 0.16.5
  • Python version used (python -V): 3.7
  • Operating system and version: Ubuntu
@ssakhavi ssakhavi added the Issue: Bug Report 🐞 Bug that needs to be fixed label Oct 9, 2020
@Minyus
Copy link
Contributor

Minyus commented Oct 9, 2020

I'm suffering from the same issue.
BTW, the correct version of pytorch-lightning is 0.10.0, not V1.0.

@ssakhavi
Copy link
Author

ssakhavi commented Oct 9, 2020

@Minyus
They've announced that 0.10 and 1.0 are the same.
https://github.com/PyTorchLightning/pytorch-lightning/releases/tag/0.10.0

@Minyus
Copy link
Contributor

Minyus commented Oct 9, 2020

@ssakhavi
I see. Sorry, I just checked PyPI and not the release note.

@Minyus
Copy link
Contributor

Minyus commented Oct 9, 2020

If QB team doesn't want users to use fsspec >= 0.7.0, how about showing just a warning instead of setting upper limit in requirements.txt?

@ssakhavi
Copy link
Author

ssakhavi commented Oct 9, 2020

To be honest I don't know how much of a breaking change the newer fsspec introduces to Kedro.

Maybe someone from the team can elaborate.

@crypdick
Copy link

crypdick commented Oct 12, 2020

@ssakhavi you may want to see #489 , according to #489 (comment) the fix is already in develop branch and will be rolled out in v0.17.0

@ssakhavi
Copy link
Author

@crypdick excellent. Thanks.

@Minyus
Copy link
Contributor

Minyus commented Nov 4, 2020

@crypdick
@ssakhavi

As of 4th Nov 2020, https://github.com/quantumblacklabs/kedro/blob/develop/requirements.txt#L5 sets fsspec>=0.5.1, <0.8, which means that current develop branch does not support fsspec >= 0.8 and cannot solve this PyTorch Lightning issue.

@lorenabalan
Copy link
Contributor

lorenabalan commented Nov 19, 2020

Hi guys, we've updated the fsspec requirements to include 0.8.x and it will be made available in kedro 0.17.0. Thank you very much for your patience with this! Relevant commit: 151cbcd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Bug Report 🐞 Bug that needs to be fixed
Projects
None yet
Development

No branches or pull requests

4 participants