Skip to content

Fix maximum version limit for mpmath#6482

Merged
pavoljuhas merged 1 commit intoquantumlib:mainfrom
pavoljuhas:fix-mpmath-requirement
Feb 28, 2024
Merged

Fix maximum version limit for mpmath#6482
pavoljuhas merged 1 commit intoquantumlib:mainfrom
pavoljuhas:fix-mpmath-requirement

Conversation

@pavoljuhas
Copy link
Copy Markdown
Collaborator

The 1.3.* pattern is invalid in install_requires context.
The mpmath<1.4 spec prohibits pre-releases of 1.4 such as 1.4.0a0.

Related to #6475

The `1.3.*` pattern is invalid in `install_requires` context.
The `mpmath<1.4` spec prohibits pre-releases of 1.4 such as 1.4.0a0.

Related to quantumlib#6475
@pavoljuhas pavoljuhas requested review from a team, cduck and vtomole as code owners February 28, 2024 07:46
@CirqBot CirqBot added the Size: XS <10 lines changed label Feb 28, 2024
@pavoljuhas
Copy link
Copy Markdown
Collaborator Author

Fixes packaging related CI failures due to invalid requirement introduced in #6477.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.74%. Comparing base (02d29a9) to head (f2a2789).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6482      +/-   ##
==========================================
- Coverage   97.75%   97.74%   -0.01%     
==========================================
  Files        1105     1105              
  Lines       94925    94925              
==========================================
- Hits        92793    92788       -5     
- Misses       2132     2137       +5     

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

@kenya-sk
Copy link
Copy Markdown
Contributor

@pavoljuhas
Thank you for your assistance.

Copy link
Copy Markdown
Collaborator

@vtomole vtomole left a comment

Choose a reason for hiding this comment

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

LGTM

@pavoljuhas pavoljuhas merged commit 78f05f4 into quantumlib:main Feb 28, 2024
@pavoljuhas
Copy link
Copy Markdown
Collaborator Author

@pavoljuhas Thank you for your assistance.

sure, my wrong in the first place for not checking the initial expression.

@pavoljuhas pavoljuhas deleted the fix-mpmath-requirement branch February 28, 2024 17:35
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
The `1.3.*` pattern is invalid in `install_requires` context.
The `mpmath<1.4` spec prohibits pre-releases of 1.4 such as 1.4.0a0.

Related to quantumlib#6475
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants