Skip to content

build: get torch from PyPI except as dev dependency on Windows#717

Merged
lars-reimann merged 1 commit intomainfrom
torch-cuda-as-dev-deps
May 4, 2024
Merged

build: get torch from PyPI except as dev dependency on Windows#717
lars-reimann merged 1 commit intomainfrom
torch-cuda-as-dev-deps

Conversation

@lars-reimann
Copy link
Member

@lars-reimann lars-reimann commented May 4, 2024

Summary of Changes

  • For MacOS, get torch from PyPI. This fixes an installation error.
  • For Linux, get torch from PyPI. It already includes CUDA support.
  • For Windows, get torch from https://download.pytorch.org/whl/cu121, but only as a dev dependency. Anyone using the library, always gets the default torch version without CUDA from PyPI, unless they explicitly change that. The source defined in pyproject.toml is completely ignored.

@github-actions
Copy link
Contributor

github-actions bot commented May 4, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ REPOSITORY git_diff yes no 0.21s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@codecov
Copy link

codecov bot commented May 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (46f54af) to head (db29faf).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #717   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           66        66           
  Lines         4811      4811           
=========================================
  Hits          4811      4811           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lars-reimann lars-reimann marked this pull request as ready for review May 4, 2024 14:23
@lars-reimann lars-reimann merged commit f2f4418 into main May 4, 2024
@lars-reimann lars-reimann deleted the torch-cuda-as-dev-deps branch May 4, 2024 14:24
@lars-reimann
Copy link
Member Author

🎉 This PR is included in version 0.23.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lars-reimann lars-reimann added the released Included in a release label May 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Included in a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant