-
Notifications
You must be signed in to change notification settings - Fork 5
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
requirements #2
Comments
We had the same issue. We cannot replicate it as we are running against several issues of pytorch and do not know which versions were used. |
We made it. In train.py you have to delete the line Then in module.py change line 96 so that the parameter is called labels instead of lm_labels thus: |
In line 79 of train.py, the author mentioned pytorch_lightning 0.8.4. With torch==1.7.1, we have another working combination. I have also rename lm_labels to labels in module.py. But now, I have two problems. 1. After training for 25 epochs, the success rate on dev dataset is still below 10%. 2. When I load the trained ckpt (from https://onebigdatabag.blob.core.windows.net/shared/base_gated_e09_0.02626.ckpt), the following error pops out which seems like a pytorch_lightning issue. ~/.local/lib/python3.8/site-packages/pytorch_lightning/core/saving.py in load_from_checkpoint(cls, checkpoint_path, map_location, hparams_file, tags_csv, *args, **kwargs) ~/.local/lib/python3.8/site-packages/pytorch_lightning/core/saving.py in _load_model_state(cls, checkpoint, *cls_args, **cls_kwargs) /usr/local/lib/python3.8/site-packages/torch/nn/modules/module.py in load_state_dict(self, state_dict, strict) RuntimeError: Error(s) in loading state_dict for SeqGenSQL: |
Hi,
I have been trying to reproduce the results in this repo. But it seems I am having some version issue for some packages, like pytorch_lightning. Is it possible to add a requirement file about all the packages that work together?
Thank you very much!
The text was updated successfully, but these errors were encountered: