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

MAINT/FIX: Relax version pin for symengine #498

Merged
merged 2 commits into from Nov 24, 2023

Conversation

richardotis
Copy link
Collaborator

@richardotis richardotis commented Nov 24, 2023

Relaxes the symengine version pin so that future symengine releases also satisfy the dependency requirement. SymEngine release cadence has increased and we're also seeing fewer regressions, so I think it's safe and advisable for pycalphad to relax the hard pin. Also, this will fix #497.

Checklist

  • If any dependencies have changed, the changes are reflected in the
    • setup.py (runtime requirements)
    • pyproject.toml (build requirements)
    • requirements-dev.txt (build and development requirements)

@richardotis richardotis added bug dependencies Pull requests that update a dependency file labels Nov 24, 2023
…s for older Pythons aren't being built anymore
@richardotis richardotis changed the title MAINT/FIX: Bump and relax version pin for symengine MAINT/FIX: Relax version pin for symengine Nov 24, 2023
Copy link

codecov bot commented Nov 24, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b2df71c) 90.48% compared to head (bb6b788) 90.47%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #498      +/-   ##
===========================================
- Coverage    90.48%   90.47%   -0.02%     
===========================================
  Files           50       50              
  Lines         7871     7871              
===========================================
- Hits          7122     7121       -1     
- Misses         749      750       +1     

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

Copy link
Collaborator

@bocklund bocklund left a comment

Choose a reason for hiding this comment

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

Is unpinning the best way to ensure that the next SymEngine release is breaking? 😄

LGTM

@richardotis richardotis merged commit 1563bc4 into pycalphad:develop Nov 24, 2023
25 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unexpected output for calculations on Python 3.11
2 participants