Commit
2 years ago
gh-36896: `src/sage/interfaces/{gap,kenzo,scilab}`: fix doctests warnings Fix warnings like ``` File "src/sage/interfaces/gap.py", line 1509, in sage.interfaces.gap.gap_reset_workspace Warning: Variable 'gap' referenced here was set only in doctest marked '# long time' P = [Process(target=gap, args=("14242",)) for i in range(4)] File "src/sage/interfaces/kenzo.py", line 134, in sage.interfaces.kenzo.Sphere Warning: Consider using a block-scoped tag by inserting the line 'sage: # optional - kenzo' just before this line to avoid repeating the tag 4 times from sage.interfaces.kenzo import Sphere # optional - kenzo File "src/sage/interfaces/scilab.py", line 252, in sage.interfaces.scilab.Scilab.set_seed Warning: Consider using a block-scoped tag by inserting the line 'sage: # optional - scilab' just before this line to avoid repeating the tag 4 times from sage.interfaces.scilab import Scilab # optional - scilab ``` We also use more block scope tags. However, we let one warning in `src/sage/interfaces/gp.py`. This is a doctest with `# 32-bit` and `# 64-bit` tags. ``` File "src/sage/interfaces/gp.py", line 799, in sage.interfaces.gp.Gp.new_with_bits_prec Warning: Variable 'pi_def' referenced here was set only in doctest marked '# needs sage.libs.pari sage.symbolic' pi_def.precision() ``` ### :memo: Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [x] The title is concise, informative, and self-explanatory. - [x] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### :hourglass: Dependencies <!-- List all open PRs that this PR logically depends on - #12345: short description why this is a dependency - #34567: ... --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: https://github.com/sagemath/sage/pull/36896 Reported by: David Coudert Reviewer(s): Matthias Köppe
Author
Release Manager
Loading