Random test failure in src/sage/plot/plot.py with exclude + detect_poles
#39002
Labels
Comments
5 tasks
vbraun
pushed a commit
to vbraun/sage
that referenced
this issue
Dec 6, 2024
…construction
As in the title.
Note that currently `ZZ(str, base=int)` only support 2 to 36. Should
this be changed for consistency?
### 📝 Checklist
<!-- Put an `x` in all the boxes that apply. -->
- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [x] I have created tests covering the changes.
- [x] I have updated the documentation and checked the documentation
preview. (There's no change in documentation.)
### ⌛ Dependencies
<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
------
The test on Conda (Mac) fails with
```
2024-11-19T04:02:36.0696760Z ./bootstrap: line 142: aclocal: command not
found
2024-11-19T04:02:36.0697900Z Bootstrap failed. Either install autotools;
or run bootstrap with
2024-11-19T04:02:36.0698610Z the -d option to download the auto-
generated files instead.
```
Looks unrelated.
Another one (Ubuntu 3.9) is
```
2024-11-19T04:26:04.0471925Z
**********************************************************************
2024-11-19T04:26:04.0591101Z File "src/sage/plot/plot.py", line 1857, in
sage.plot.plot.plot
2024-11-19T04:26:04.0591946Z Failed example:
2024-11-19T04:26:04.0592761Z plot(f, (x, -3.5, 3.5),
detect_poles='show', exclude=[-3..3],
2024-11-19T04:26:04.0593588Z ymin=-5, ymax=5)
2024-11-19T04:26:04.0594061Z Expected:
2024-11-19T04:26:04.0649808Z Graphics object consisting of 12
graphics primitives
2024-11-19T04:26:04.0650471Z Got:
2024-11-19T04:26:04.1009273Z Graphics object consisting of 13
graphics primitives
2024-11-19T04:26:26.6704957Z
**********************************************************************
```
Also looks unrelated. Reported in sagemath#39002.
URL: sagemath#39001
Reported by: user202729
Reviewer(s): grnx, Travis Scrimshaw
vbraun
pushed a commit
to vbraun/sage
that referenced
this issue
Dec 8, 2024
…construction
As in the title.
Note that currently `ZZ(str, base=int)` only support 2 to 36. Should
this be changed for consistency?
### 📝 Checklist
<!-- Put an `x` in all the boxes that apply. -->
- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [x] I have created tests covering the changes.
- [x] I have updated the documentation and checked the documentation
preview. (There's no change in documentation.)
### ⌛ Dependencies
<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
------
The test on Conda (Mac) fails with
```
2024-11-19T04:02:36.0696760Z ./bootstrap: line 142: aclocal: command not
found
2024-11-19T04:02:36.0697900Z Bootstrap failed. Either install autotools;
or run bootstrap with
2024-11-19T04:02:36.0698610Z the -d option to download the auto-
generated files instead.
```
Looks unrelated.
Another one (Ubuntu 3.9) is
```
2024-11-19T04:26:04.0471925Z
**********************************************************************
2024-11-19T04:26:04.0591101Z File "src/sage/plot/plot.py", line 1857, in
sage.plot.plot.plot
2024-11-19T04:26:04.0591946Z Failed example:
2024-11-19T04:26:04.0592761Z plot(f, (x, -3.5, 3.5),
detect_poles='show', exclude=[-3..3],
2024-11-19T04:26:04.0593588Z ymin=-5, ymax=5)
2024-11-19T04:26:04.0594061Z Expected:
2024-11-19T04:26:04.0649808Z Graphics object consisting of 12
graphics primitives
2024-11-19T04:26:04.0650471Z Got:
2024-11-19T04:26:04.1009273Z Graphics object consisting of 13
graphics primitives
2024-11-19T04:26:26.6704957Z
**********************************************************************
```
Also looks unrelated. Reported in sagemath#39002.
URL: sagemath#39001
Reported by: user202729
Reviewer(s): grnx, Travis Scrimshaw
vbraun
pushed a commit
to vbraun/sage
that referenced
this issue
Dec 8, 2024
…construction
As in the title.
Note that currently `ZZ(str, base=int)` only support 2 to 36. Should
this be changed for consistency?
### 📝 Checklist
<!-- Put an `x` in all the boxes that apply. -->
- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [x] I have created tests covering the changes.
- [x] I have updated the documentation and checked the documentation
preview. (There's no change in documentation.)
### ⌛ Dependencies
<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
------
The test on Conda (Mac) fails with
```
2024-11-19T04:02:36.0696760Z ./bootstrap: line 142: aclocal: command not
found
2024-11-19T04:02:36.0697900Z Bootstrap failed. Either install autotools;
or run bootstrap with
2024-11-19T04:02:36.0698610Z the -d option to download the auto-
generated files instead.
```
Looks unrelated.
Another one (Ubuntu 3.9) is
```
2024-11-19T04:26:04.0471925Z
**********************************************************************
2024-11-19T04:26:04.0591101Z File "src/sage/plot/plot.py", line 1857, in
sage.plot.plot.plot
2024-11-19T04:26:04.0591946Z Failed example:
2024-11-19T04:26:04.0592761Z plot(f, (x, -3.5, 3.5),
detect_poles='show', exclude=[-3..3],
2024-11-19T04:26:04.0593588Z ymin=-5, ymax=5)
2024-11-19T04:26:04.0594061Z Expected:
2024-11-19T04:26:04.0649808Z Graphics object consisting of 12
graphics primitives
2024-11-19T04:26:04.0650471Z Got:
2024-11-19T04:26:04.1009273Z Graphics object consisting of 13
graphics primitives
2024-11-19T04:26:26.6704957Z
**********************************************************************
```
Also looks unrelated. Reported in sagemath#39002.
URL: sagemath#39001
Reported by: user202729
Reviewer(s): grnx, Travis Scrimshaw
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Steps To Reproduce
See https://github.com/sagemath/sage/actions/runs/11905966286/job/33177274834?pr=39001 .
A pickled representation of the graphics object can be found here. Basically the problem appears to be somehow SageMath detect that there is a pole or something at 0 and draw a vertical dotted line there (which is the primitive
g[6]).(reproduce with
G = Graphics(); G.add_primitive(o[6]); G.axes(False); G)The text was updated successfully, but these errors were encountered: