Migrate "long" ci tests to Meson#40158
Conversation
|
Technically this "works" too (if it works properly). Let's see. |
.github/workflows/ci-meson.yml
Outdated
| python: '3.13' | ||
| tests: 'all' | ||
| exclude: | ||
| # Exclude 'all' since we already have 'long' |
There was a problem hiding this comment.
maybe it's time to consider having a more descriptive name. new (both short and long), short, long (this should really be all, but that's doubly confusing because all currently means all short)?
(if you mean |
| sage/graphs/graph_plot.py | ||
| sage/misc/sagedoc.py | ||
|
|
||
| [coverage:run] |
There was a problem hiding this comment.
is there anything else that still uses tox?
|
the Maybe try changing it to |
Do you understand what's going on here? The queue has a max limit of 1, because it is supposed to hold only one doctest result. I don't see why it would try to put more than one result in there. The code also doesn't seem to reuse the queue across workers/doctests. |
no idea either, but some StackOverflow post claims this may be caused when the object being passed around is too large (I can't reproduce this on my side though). In that case it sounds like blocking=True might help. Worst case we can add some debugging there (print out what's the current length), modulo race conditions. |
it's not helping either. I'll see if I can reproduce the issue locally… edit: I cannot. debugging with #40161 edit: feels like a weird bug in Now continue debugging. |
|
sanity check, is test-new currently running long tests? (it's supposed to be.) |
|
Documentation preview for this PR (built with commit a5aa2b4; changes) is ready! 🎉 |
|
Needs #39872 to be fixed. |
Fixes #40120 and Fixes #39871.
Sage-the-distro is still tested using ci-incremental.
📝 Checklist
⌛ Dependencies