Skip to content

Conversation

@goxq
Copy link
Collaborator

@goxq goxq commented Oct 15, 2025

What does this PR do?

Update Python version in README for conda setup and change torch dependency to flexible version.

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Was this discussed/approved via a GitHub issue? Please add a link to it if that's the case.
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

Copilot AI review requested due to automatic review settings October 15, 2025 05:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Python version requirement and makes the torch dependency more flexible. The changes address version compatibility and installation flexibility for the project.

  • Updates Python version from 3.10 to 3.11 in conda environment setup instructions
  • Removes pinned torch version constraint, allowing any version to be installed

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Removes specific torch version constraint to allow flexible version selection
README.md Updates Python version from 3.10 to 3.11 in conda environment creation command

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

"wandb>=0.20.0",

"torch==2.7.0",
"torch",
Copy link

Copilot AI Oct 15, 2025

Choose a reason for hiding this comment

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

Removing the version constraint for torch could lead to compatibility issues. Consider using a minimum version constraint like 'torch>=2.0.0' to ensure compatibility while maintaining flexibility.

Suggested change
"torch",
"torch>=2.0.0",

Copilot uses AI. Check for mistakes.
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