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

[BUG] error with pytorch_lightning import #1732

Closed
cbasisarifin opened this issue Dec 18, 2024 · 9 comments
Closed

[BUG] error with pytorch_lightning import #1732

cbasisarifin opened this issue Dec 18, 2024 · 9 comments
Labels
bug Something isn't working

Comments

@cbasisarifin
Copy link

Describe the bug

To Reproduce

i notice that the error came from "pytorch_forecasting\models\deepar\__init__.py" line 13
is the line "from pytorch_lightning.core.lightning import LightningModule" in line 13 can be change into
"from lightning import LightningModule"

are they both the same module?

Expected behavior

Additional context

Versions

@cbasisarifin cbasisarifin added the bug Something isn't working label Dec 18, 2024
@github-project-automation github-project-automation bot moved this to Needs triage & validation in Bugfixing - pytorch-forecasting Dec 18, 2024
@fkiraly
Copy link
Collaborator

fkiraly commented Dec 20, 2024

Can you please add code to reproduce the error?

@fkiraly fkiraly changed the title [BUG] [BUG] error wit Dec 20, 2024
@fkiraly fkiraly changed the title [BUG] error wit [BUG] error with pytorch_lightning support Dec 20, 2024
@fkiraly fkiraly changed the title [BUG] error with pytorch_lightning support [BUG] error with pytorch_lightning import Dec 20, 2024
@benHeid
Copy link
Collaborator

benHeid commented Dec 24, 2024

pytorch-lightning is the old name. of lightning. Thus, only lightning should be used. However, in line 13 of the mentioned __init__ I do not see any import of lightning. However, there are multiple old links pointing to pytorch-lightning. These links are probably broken and we should replace them.

@fkiraly
Copy link
Collaborator

fkiraly commented Dec 26, 2024

why are the tests not catching this?

@benHeid
Copy link
Collaborator

benHeid commented Dec 26, 2024

Because they are part of the docstrings. Do we test those links?

@cbasisarifin
Copy link
Author

Pytorch-forecasting 0 10 2
image

above is the version and the example screen shot after altered within folder "\site-packages\pytorch_forecasting\models\deepar"

image
above is the screenshot before altered

@benHeid
Copy link
Collaborator

benHeid commented Dec 26, 2024

This is an old version of PyTorch forecasting. Please try to update it.

@cbasisarifin
Copy link
Author

thank you very much for the reply
however, i just copy and pasted the command directly from the webpage
image

i tried to upgrade them from Conda-forge directly,
but still not success, used chatgpt to guide me how to upgrade it, even worsen, got downgraded into version 0.5.3

and then chatgpt request me to check the version available in the conda forge

and this is the respon

image

is it safe to directly upgrade version via pip?

regards,
Chandra

@benHeid
Copy link
Collaborator

benHeid commented Dec 27, 2024

Oh try to install it with pip. @fkiraly do you know what's with the version on conda?

@cbasisarifin
Copy link
Author

anyway
thanks for replying @benHeid and @fkiraly
happy new year 2025 wish you guys luck and healthy

anyway
just want to directly answer the last comment
image

but, i've manually install it via pip
now i will close the thread
thanks again @benHeid and @fkiraly
regards,
Chandra

@github-project-automation github-project-automation bot moved this from Needs triage & validation to Fixed/resolved in Bugfixing - pytorch-forecasting Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Fixed/resolved
Development

No branches or pull requests

3 participants