sage.doctest.control: In doctests, use Python floats for stats#38122
sage.doctest.control: In doctests, use Python floats for stats#38122vbraun merged 1 commit intosagemath:developfrom
sage.doctest.control: In doctests, use Python floats for stats#38122Conversation
…he stats dictionary, not Sage RealNumbers
|
Documentation preview for this PR (built with commit 3947d9a; changes) is ready! 🎉 |
|
With this (plus #38081),
(I am also still seeing errors as reported in #35646, but I've edited those out. I haven't seen all tests pass on an OS X machine in months.) |
|
I have these environment variables set: I don't think I set anything else that could be relevant. |
|
If I unset |
|
Do you still see any of those "JSON" errors? |
|
What is |
|
Tests pass with and |
No. |
|
OK, then I think the failures that show as "Doctest interrupted" are a separate issue. |
|
Actually with |
|
Thanks! |
sagemathgh-38122: `sage.doctest.control`: In doctests, use Python floats for stats <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Hopefully fixes sagemath#38116 @jhpalmieri ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#38122 Reported by: Matthias Köppe Reviewer(s):
sagemathgh-38122: `sage.doctest.control`: In doctests, use Python floats for stats <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Hopefully fixes sagemath#38116 @jhpalmieri ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#38122 Reported by: Matthias Köppe Reviewer(s):
sagemathgh-38122: `sage.doctest.control`: In doctests, use Python floats for stats <!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> Hopefully fixes sagemath#38116 @jhpalmieri ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [ ] The description explains in detail what this PR is about. - [x] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation and checked the documentation preview. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on. For example, --> <!-- - sagemath#12345: short description why this is a dependency --> <!-- - sagemath#34567: ... --> URL: sagemath#38122 Reported by: Matthias Köppe Reviewer(s):
Hopefully fixes #38116 @jhpalmieri
📝 Checklist
⌛ Dependencies