-
-
Notifications
You must be signed in to change notification settings - Fork 528
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix some doctest warnings #37279
Fix some doctest warnings #37279
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the CI indicates a doctest warning in src/sage/geometry/polyhedron/backend_field.py
line 93 that could be fixed in this PR: Variable 'p' referenced here was set only in doctest marked '# needs sage.rings.number_field sage.symbolic'
Indeed, thanks. |
Documentation preview for this PR (built with commit 9bfc29a; changes) is ready! 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Thank you! |
These are some of the last remaining doctest warnings as of 10.3.beta7, as seen e.g. in https://github.com/sagemath/sage/pull/37250/files "Unchanged files with check annotations".
The ones not fixed here will need a solution for:
# 32-bit
/# 64-bit
tag breaks block-scoped doctest tags #36099📝 Checklist
⌛ Dependencies