8000 Pin `sympy >= 1.13.0` by XuehaiPan · Pull Request #130895 · pytorch/pytorch · GitHub
[go: up one dir, main page]

Skip to content

Pin sympy >= 1.13.0 #130895

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

Closed
wants to merge 19 commits into from
Closed

Conversation

XuehaiPan
Copy link
Collaborator
@XuehaiPan XuehaiPan commented Jul 17, 2024

Stack from ghstack (oldest at bottom):


The opposite of #130836. Pin sympy >= 1.13.0 for Python >= 3.9 and sympy == 1.12.1 for Python 3.8.

See the PR description of #130836 for more details.

sympy 1.13.0 introduces some breaking changes which break our tests. More specifically:

BREAKING CHANGE: Float and Integer/Rational no longer compare equal with a == b. From now on Float(2.0) != Integer(2). Previously expressions involving Float would compare unequal e.g. x2.0 != x2 but an individual Float would compare equal to an Integer. In SymPy 1.7 a Float will always compare unequal to an Integer even if they have the same "value". Use sympy.numbers.int_valued(number) to test if a number is a concrete number with no decimal part. (#25614 by @smichr)

sympy >= 1.13.0 is required to enable Python 3.13 support. This should be part of #130689.

cc @jgong5 @mingfeima @XiaobingSuper @sanchitintel @ashokei @jingxu10 @seemethere @malfet @pytorch/pytorch-dev-infra @voznesenskym @penguinwu @EikanWang @Guobing-Chen @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @peterbell10 @ipiszy @yf225 @chenyang78 @kadeng @muchulee8 @ColinPeppler @amjames @desertfire @chauhang

[ghstack-poisoned]
@XuehaiPan XuehaiPan requested a review from a team as a code owner July 17, 2024 03:05
Copy link
pytorch-bot bot commented Jul 17, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/130895

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 Cancelled Job

As of commit ef9ca93 with merge base 3c622fb (image):

CANCELLED JOB - The following job was cancelled. Please retry:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Jul 17, 2024
@XuehaiPan XuehaiPan added module: ci Related to continuous integration ciflow/trunk Trigger trunk jobs on your pull request release notes: releng release notes category ciflow/mps Run MPS tests (subset of trunk) ciflow/rocm Trigger "default" config CI on ROCm labels Jul 17, 2024
@XuehaiPan XuehaiPan marked this pull request as draft July 17, 2024 03:09
@XuehaiPan XuehaiPan requested review from albanD and ezyang July 17, 2024 03:09
[ghstack-poisoned]
@pytorch-bot pytorch-bot bot added ciflow/inductor module: cpu CPU specific problem (e.g., perf, algorithm) module: inductor labels Jul 17, 2024
@XuehaiPan
Copy link
Collaborator Author
XuehaiPan commented Jul 17, 2024

Found another bug for sympy:

In [1]: import sympy

In [2]: f = sympy.Float(0.0)

In [3]: abs(f)
Out[3]: 0

In [4]: f == abs(f)
Out[4]: False

[ghstack-poisoned]
[ghstack-poisoned]
XuehaiPan added a commit that referenced this pull request Jul 17, 2024
ghstack-source-id: ac1dd3b
Pull Request resolved: #130895
@XuehaiPan XuehaiPan marked this pull request as ready for review July 17, 2024 06:24
[ghstack-poisoned]
@XuehaiPan
Copy link
Collaborator Author

@pytorchbot merge -i

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged while ignoring the following 1 checks: Mac MPS / macos-py3-arm64-mps / test (mps, 1, 1, macos-m2-14)

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

pytorchmergebot pushed a commit that referenced this pull request Jul 20, 2024
0.12.0 Major Updates:

- Add context manager to temporarily set the dictionary sorting mode
- Add accessor APIs
- Use `stable` tag for `pybind11` for Python 3.13 support
- Fix potential segmentation fault for pickling support

0.12.1 Updates:

- Fix warning regression during import when launch with strict warning filters

Closes #130155
Pull Request resolved: #130139
Approved by: https://github.com/zou3519
ghstack dependencies: #130895
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Jul 20, 2024
------

The opposite of pytorch#130836. Pin `sympy >= 1.13.0` for Python >= 3.9 and `sympy == 1.12.1` for Python 3.8.

- pytorch#130836

See the PR description of pytorch#130836 for more details.

`sympy` 1.13.0 introduces some breaking changes which break our tests. More specifically:

- Ref [Backwards compatibility breaks and deprecations](https://github.com/sympy/sympy/wiki/release-notes-for-1.13.0#backwards-compatibility-breaks-and-deprecations)

> BREAKING CHANGE: Float and Integer/Rational no longer compare equal with a == b. From now on Float(2.0) != Integer(2). Previously expressions involving Float would compare unequal e.g. x*2.0 != x*2 but an individual Float would compare equal to an Integer. In SymPy 1.7 a Float will always compare unequal to an Integer even if they have the same "value". Use sympy.numbers.int_valued(number) to test if a number is a concrete number with no decimal part. ([pytorch#25614](sympy/sympy#25614) by [@smichr](https://github.com/smichr))

`sympy >= 1.13.0` is required to enable Python 3.13 support. This should be part of pytorch#130689.

- pytorch#130689

Pull Request resolved: pytorch#130895
Approved by: https://github.com/ezyang
XuehaiPan added a commit to XuehaiPan/pytorch that referenced this pull request Jul 21, 2024
------

The opposite of pytorch#130836. Pin `sympy >= 1.13.0` for Python >= 3.9 and `sympy == 1.12.1` for Python 3.8.

- pytorch#130836

See the PR description of pytorch#130836 for more details.

`sympy` 1.13.0 introduces some breaking changes which break our tests. More specifically:

- Ref [Backwards compatibility breaks and deprecations](https://github.com/sympy/sympy/wiki/release-notes-for-1.13.0#backwards-compatibility-breaks-and-deprecations)

> BREAKING CHANGE: Float and Integer/Rational no longer compare equal with a == b. From now on Float(2.0) != Integer(2). Previously expressions involving Float would compare unequal e.g. x*2.0 != x*2 but an individual Float would compare equal to an Integer. In SymPy 1.7 a Float will always compare unequal to an Integer even if they have the same "value". Use sympy.numbers.int_valued(number) to test if a number is a concrete number with no decimal part. ([pytorch#25614](sympy/sympy#25614) by [@smichr](https://github.com/smichr))

`sympy >= 1.13.0` is required to enable Python 3.13 support. This should be part of pytorch#130689.

- pytorch#130689

Pull Request resolved: pytorch#130895
Approved by: https://github.com/ezyang
pytorchmergebot pushed a commit that referenced this pull request Jul 21, 2024
DiweiSun pushed a commit to DiweiSun/pytorch that referenced this pull request Jul 22, 2024
------

The opposite of pytorch#130836. Pin `sympy >= 1.13.0` for Python >= 3.9 and `sympy == 1.12.1` for Python 3.8.

- pytorch#130836

See the PR description of pytorch#130836 for more details.

`sympy` 1.13.0 introduces some breaking changes which break our tests. More specifically:

- Ref [Backwards compatibility breaks and deprecations](https://github.com/sympy/sympy/wiki/release-notes-for-1.13.0#backwards-compatibility-breaks-and-deprecations)

> BREAKING CHANGE: Float and Integer/Rational no longer compare equal with a == b. From now on Float(2.0) != Integer(2). Previously expressions involving Float would compare unequal e.g. x*2.0 != x*2 but an individual Float would compare equal to an Integer. In SymPy 1.7 a Float will always compare unequal to an Integer even if they have the same "value". Use sympy.numbers.int_valued(number) to test if a number is a concrete number with no decimal part. ([pytorch#25614](sympy/sympy#25614) by [@smichr](https://github.com/smichr))

`sympy >= 1.13.0` is required to enable Python 3.13 support. This should be part of pytorch#130689.

- pytorch#130689

Pull Request resolved: pytorch#130895
Approved by: https://github.com/ezyang
DiweiSun pushed a commit to DiweiSun/pytorch that referenced this pull request Jul 22, 2024
0.12.0 Major Updates:

- Add context manager to temporarily set the dictionary sorting mode
- Add accessor APIs
- Use `stable` tag for `pybind11` for Python 3.13 support
- Fix potential segmentation fault for pickling support

0.12.1 Updates:

- Fix warning regression during import when launch with strict warning filters

Closes pytorch#130155
Pull Request resolved: pytorch#130139
Approved by: https://github.com/zou3519
ghstack dependencies: pytorch#130895
DiweiSun pushed a commit to DiweiSun/pytorch that referenced this pull request Jul 22, 2024
@albanD
Copy link
Collaborator
albanD commented Jul 22, 2024

@XuehaiPan this is actually breaking nighlty builds on conda because only sympy up to 1.12.0 seems to be available there: https://anaconda.org/anaconda/sympy/files
Do you know if it is expected?

@XuehaiPan
Copy link
Collaborator Author

on conda because only sympy up to 1.12.0 seems to be available there: anaconda.org/anaconda/sympy/files
Do you know if it is expected?

@albanD We may need to open an issue here (https://github.com/ContinuumIO/anaconda-issues/issues?q=label%3Apackage_request+) to request a version update.

The short-term solution is to loosen the constraint to sympy >= 1.12.0 (1.12 is required for function equal_valued).

pytorchmergebot pushed a commit that referenced this pull request Jul 23, 2024
XuehaiPan pushed a commit to XuehaiPan/pytorch that referenced this pull request Jul 23, 2024
XuehaiPan pushed a commit to XuehaiPan/pytorch that referenced this pull request Jul 24, 2024
xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Jul 25, 2024
------

The opposite of pytorch#130836. Pin `sympy >= 1.13.0` for Python >= 3.9 and `sympy == 1.12.1` for Python 3.8.

- pytorch#130836

See the PR description of pytorch#130836 for more details.

`sympy` 1.13.0 introduces some breaking changes which break our tests. More specifically:

- Ref [Backwards compatibility breaks and deprecations](https://github.com/sympy/sympy/wiki/release-notes-for-1.13.0#backwards-compatibility-breaks-and-deprecations)

> BREAKING CHANGE: Float and Integer/Rational no longer compare equal with a == b. From now on Float(2.0) != Integer(2). Previously expressions involving Float would compare unequal e.g. x*2.0 != x*2 but an individual Float would compare equal to an Integer. In SymPy 1.7 a Float will always compare unequal to an Integer even if they have the same "value". Use sympy.numbers.int_valued(number) to test if a number is a concrete number with no decimal part. ([pytorch#25614](sympy/sympy#25614) by [@smichr](https://github.com/smichr))

`sympy >= 1.13.0` is required to enable Python 3.13 support. This should be part of pytorch#130689.

- pytorch#130689

Pull Request resolved: pytorch#130895
Approved by: https://github.com/ezyang
xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Jul 25, 2024
0.12.0 Major Updates:

- Add context manager to temporarily set the dictionary sorting mode
- Add accessor APIs
- Use `stable` tag for `pybind11` for Python 3.13 support
- Fix potential segmentation fault for pickling support

0.12.1 Updates:

- Fix warning regression during import when launch with strict warning filters

Closes pytorch#130155
Pull Request resolved: pytorch#130139
Approved by: https://github.com/zou3519
ghstack dependencies: pytorch#130895
xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Jul 25, 2024
xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Jul 25, 2024
@github-actions github-actions bot deleted the gh/XuehaiPan/126/head branch August 23, 2024 01:59
@adamjstewart
Copy link
Contributor

Is there anything we can do to relax this version pin? Would make compatibility in large environments easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/inductor ciflow/mps Run MPS tests (subset of trunk) ciflow/rocm Trigger "default" config CI on ROCm ciflow/trunk Trigger trunk jobs on your pull request keep-going Don't stop on first failure, keep running tests until the end Merged module: ci Related to continuous integration module: cpu CPU specific problem (e.g., perf, algorithm) module: inductor open source release notes: releng release notes category
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

8 participants
0