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

优化对torch版本的依赖声明为>=1.12.1且<2.0 #27

Merged
merged 1 commit into from
Dec 21, 2022

Conversation

Starrah
Copy link
Member

@Starrah Starrah commented Dec 21, 2022

否则,现在的写法是要求用户的pytorch版本严格等于1.12.1,即使用户已经安装了更新的版本也是不行的。我认为应当允许用户使用任何Pytorch 1.X(只要大于某个特定版本,因为我们的程序可能会使用某些较新的Pytorch特性)。
根据语义化版本规则,相同的大版本号一般意味着没有API Breaking Changes,因此这样做应当不会产生问题,同时也给了用户更大的自由。
PS:版本声明字符串的语法参考https://python-poetry.org/docs/dependency-specification/#multiple-requirements

否则,现在的写法是要求用户的pytorch版本严格等于1.12.1,即使用户已经安装了更新的版本也是不行的。我认为应当允许用户使用任何Pytorch 1.X(只要大于某个特定版本,因为我们的程序可能会使用某些较新的Pytorch特性)。
根据语义化版本规则,相同的大版本号一般意味着没有API Breaking Changes,因此这样做应当不会产生问题,同时也给了用户更大的自由。
PS:版本声明字符串的语法参考https://python-poetry.org/docs/dependency-specification/#multiple-requirements
@Starrah Starrah requested a review from yifanfeng97 December 21, 2022 12:17
@yifanfeng97 yifanfeng97 merged commit 2b25d32 into iMoonLab:main Dec 21, 2022
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