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

CI failures due to TypeError: cannot inherit frozen dataclass from a non-frozen one #1524

Closed
lbianchi-lbl opened this issue Nov 7, 2024 · 5 comments
Assignees
Labels
dependencies Pull requests that update a dependency file Priority:High High Priority Issue or PR

Comments

@lbianchi-lbl
Copy link
Contributor

lbianchi-lbl commented Nov 7, 2024

  • Something within the past few hours must have happened causing the CI to fail:
    image
  • This points towards the cause being a dependency being updated (as opposed to a change in the code, since there were no commits merged between the passing and failing CI workflow runs below):
    image
  • However, at first sight, none of the packages involved in the stack trace have been updated in that timeframe:
    • Pyomo 6.8.1a0 (released a few weeks ago)
    • Pint 0.24.3 (released July 2024)
    • dataclasses is part of the Python standard library, so unlikely to have been changed in a backward-incompatible way within the same Python version

Related

We're aware of this error affecting the following PRs

@lbianchi-lbl lbianchi-lbl self-assigned this Nov 7, 2024
@lbianchi-lbl
Copy link
Contributor Author

From comparing pip list between the passing (LHS) and failing (RHS) runs, there are two differences:
image
image

@lbianchi-lbl
Copy link
Contributor Author

It looks like flexparser is the likely culprit, see hgrecco/pint#2074

@blnicho
Copy link
Member

blnicho commented Nov 7, 2024

@lbianchi-lbl the issue is the latest flexparser not being compatible with pint. See the discussion here: hgrecco/pint#1969

I've temporarily pinned flexparser in #1523

@blnicho
Copy link
Member

blnicho commented Nov 7, 2024

New pint release is out: https://github.com/hgrecco/pint/releases/tag/0.24.4

@lbianchi-lbl
Copy link
Contributor Author

New pint release is out: https://github.com/hgrecco/pint/releases/tag/0.24.4

This has been fixed upstream, so I'm closing this issue.

@lbianchi-lbl lbianchi-lbl added Priority:High High Priority Issue or PR dependencies Pull requests that update a dependency file labels Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Priority:High High Priority Issue or PR
Projects
None yet
Development

No branches or pull requests

2 participants