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

Fix custom ops bug for pytorch 1.12 and onwards #36

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

whatsnewsisyphus
Copy link

The code does not work with any available stable version of pytorch, meaning it breaks with colab out of box. The fix is simple, you should pull it from the official repo since you are downstream from that, it's their latest commit

From: NVlabs@c233a91

Adapt to newer _jit_get_operation API that changed in pytorch/pytorch#76814

for NVlabs#188, NVlabs#193

The code does not work with any available stable version of pytorch, meaning it breaks with colab out of box. The fix is simple:

From: NVlabs@c233a91

Adapt to newer _jit_get_operation API that changed in
pytorch/pytorch#76814

for NVlabs#188, NVlabs#193
- Switched from jpeg to png as compression artifacts were becoming part of the projection.
- Improved description of projection and pytorch seed variables
- added save_n_step argument to allow a predetermined number of progress steps rather than each and every one of them
- added/tweaked a few debug/process printouts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant