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

update pytorch on apple silicon to 2.0.1 #2139

Closed

Conversation

stonerl
Copy link
Collaborator

@stonerl stonerl commented Aug 24, 2023

fixes #2138

@RunDevelopment
Copy link
Member

I think we should update everything to PyTorch 2.0 all at once. Right now, PyTorch could be completely broken on mac and we wouldn't know, because our CI doesn't test this.

@stonerl
Copy link
Collaborator Author

stonerl commented Aug 24, 2023

Shall I close it then?

@stonerl
Copy link
Collaborator Author

stonerl commented Aug 24, 2023

I did some basic testing, and it seems to be working fine. That doesn't replace CI, though.

image

@RunDevelopment
Copy link
Member

Shall I close it then?

Not sure.

@joeyballentine What's the state of PyTorch 2.0 again? Did we want to wait for 2.1?

Also, if we do go for 2.0 (or 2.1), we should probably add support for py 3.11 as well since that requires >=2.0.

@joeyballentine
Copy link
Member

I want to update to 3.11 and 2.0 at the same time. Ideally we'd wait until 2.1 since that comes out in October but I suppose we could also just update to that when it comes out instead of waiting.

Just give me a little time and I'll work on what's necessary for 3.11. The reason we need to do both is that 2.0 is slower than the current version we use. For Mac that might not matter, but I don't want to have a major version difference between platforms and accidentally cause issues from that

@joeyballentine
Copy link
Member

Closing in favor of #2143

@stonerl stonerl deleted the pytorch-upgrade-macos-arm64 branch August 26, 2023 18:24
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.

Black output image using 4x_NMKD-Superscale-SP_178000_G
3 participants