Skip to content

Update gotch to pytorch 2.10.0#140

Open
timleecasey wants to merge 6 commits intosugarme:masterfrom
timleecasey:tlc/upgrade-pytorch-210
Open

Update gotch to pytorch 2.10.0#140
timleecasey wants to merge 6 commits intosugarme:masterfrom
timleecasey:tlc/upgrade-pytorch-210

Conversation

@timleecasey
Copy link
Copy Markdown

Claude code to the rescue.
Upgrade from PyTorch 2.1.0 to 2.10.0.
Add MPS for Mac metal.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why flags for linux would be replaced with those for MacOS? Would be great if we could support both.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is going to take me a bit to understand your structure. I think we should support both. I have running go code supporting MPS and CPU as devices at the same time. I will try CUDA as I get the time.

There are a few things in here like this. Once I feel better about how this is done, better from a putting it in not-my-own-thing sense, then it is ok. Until then, there is going to be some back and forth, and I apologize for that now :).

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check this now. Any comments are good.

Added a make file as a way to say how to run tests.  The pytorch 2.10.0 is local.
Add architecture to make file to allow differences between the repsective ways of doing things.
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.

2 participants