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

IDAES / idaes-pse

The IDAES Process Systems Engineering Framework

https://idaes-pse.readthedocs.io/

Other

218 stars 235 forks source link

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

Closed lbianchi-lbl closed 2 weeks ago

lbianchi-lbl commented 2 weeks ago

Something within the past few hours must have happened causing the CI to fail:
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):
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

IDAES/idaes-pse#1523
watertap-org/watertap#1520

lbianchi-lbl commented 2 weeks ago

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

lbianchi-lbl commented 2 weeks ago

It looks like flexparser is the likely culprit, see https://github.com/hgrecco/pint/issues/2074

blnicho commented 2 weeks ago

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

I've temporarily pinned flexparser in #1523

blnicho commented 2 weeks ago

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

lbianchi-lbl commented 2 weeks ago

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.