12313 Fix test_manhole.py on Python 3.13rc2#12314
Conversation
…ceback is omitted.
CodSpeed Performance ReportMerging #12314 will not alter performanceComparing Summary
|
|
I've been seeing what I think is the same failure since updating to Python 3.12.6. |
|
It's possible this is a regression in CPython. I'll try to reproduce; perhaps a better way to implement it is to inspect the possibly-skipped frame and see if it should be skipped, given it's in a released version. And I guess file a bug report. |
|
Thanks for the PR I see that on I did a re-run to get latest |
adiroiban
left a comment
There was a problem hiding this comment.
Thanks for the changes and the fix.
I think that we can merge this, so that we fix trunk
It would help to create an upstream bug report and add the link as an inline code comment
|
There's a similar failure with Debian's Python 3.12.6, so maybe "less than 3.13" isn't quite the right test? |
|
Thanks for the feedback It looks like via Maybe we need to wait a bit more. I see that 3.12.6 is available via The strange thing, is that on the |
|
I bisected Python |
|
To replicate this error on 3.12.6 we need Run actions/setup-python@v5
with:
python-version: 3.12
- check-latest: false
+ check-latest: trueI have pushed a change on this PR to reproduce the error in 3.12.6 |
…with-latest-313-pre-release' into 12313-test_manholepy-is-failing-with-latest-313-pre-release
|
I updated the check to be tied to the symptoms, rather than version specific. |
Scope and purpose
Fixes #12313
I am not sure why this is necessary but it solves it.