-
Notifications
You must be signed in to change notification settings - Fork 191
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
TypeError: PyTorchPredictor.__init__() got an unexpected keyword argument 'freq' #159
Comments
please use the 0.7.0 branch and try? |
I cloned 0.7.0 branch but it has caused new error:
|
hmm that still looks like its the older stuff... |
I solved ImportError from previous comment by adding trainer.py to pts from master to 0.7.0 and editing init.py in the same folder. But I've got new error seems like #158
Any thoughts? |
right you need also now give the esitmator the diffusers scheduler... see a running example here: https://github.com/kashif/time_match/blob/main/Time-Grad-Solar.ipynb |
also don't think the trainer stuff from pts is needed as the 0.7.0 branch uses gluont's trainer |
@kashif above comments helped to move ahead, but I've got next error:
|
right cause pytorch-lighting has changed its API... so in the lightning module file of time-grad do:
and then subclass from that:
|
#159 (comment) helped to solve previous issue but now after all epochs I'm getting new error:
Seems like I need to pass |
hmm let me think i thought i fixed this issue as well... are you sure you are on the 0.7.0 branch? |
actually try: |
@kashif yes I'm on branch 0.7.0. #159 (comment) didn't help. |
so in the estimator set the device to a string:
|
That works! Thank you @kashif so much! |
Hi folks!
I'm trying to reproduce Time-Grad example notebook and getting this error:
Does anybody know solution?
Dependencies:
The text was updated successfully, but these errors were encountered: