Skip to content

Allow newer versions of PyTorch than we have verified.#39

Merged
TobyRoseman merged 3 commits into
apple:mainfrom
TobyRoseman:pytorch-future-versions
Jul 15, 2026
Merged

Allow newer versions of PyTorch than we have verified.#39
TobyRoseman merged 3 commits into
apple:mainfrom
TobyRoseman:pytorch-future-versions

Conversation

@TobyRoseman

Copy link
Copy Markdown
Contributor
  • Remove max PyTorch version for installing coreai-torch.
  • Pin PyTorch versions for testing.
  • Warn user if they have newer version of PyTorch than we have verified.

@gokulkrishna98 gokulkrishna98 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Installing pre-commit would help in fixing linting issues.

.pre-commit-config.yaml

Comment thread pyproject.toml
"pytest-xdist",
"torchaudio",
"torchvision",
"torch==2.13.0",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nit: if you plan to test on different torch versions, you could move the torch dependencies to a different dependency group and then "import" it into this group by doing { include-group = ... }

@TobyRoseman
TobyRoseman force-pushed the pytorch-future-versions branch from 09dc0c7 to f1602dd Compare July 15, 2026 00:12
@TobyRoseman

Copy link
Copy Markdown
Contributor Author

Installing pre-commit would help in fixing linting issues.

.pre-commit-config.yaml

Thanks. Done.

I didn't know about this. I've added setup instructions to the README as part of this PR.

@TobyRoseman
TobyRoseman merged commit ef1181b into apple:main Jul 15, 2026
2 checks passed
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.

4 participants