-
-
Notifications
You must be signed in to change notification settings - Fork 482
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
sage.manifolds
, sage.tensor
: Add # optional
doctest tags for modularization
#35267
sage.manifolds
, sage.tensor
: Add # optional
doctest tags for modularization
#35267
Conversation
…date=format:%Y FILE | sort -k2"
Documentation preview for this PR is ready! 🎉 |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #35267 +/- ##
===========================================
+ Coverage 88.57% 88.58% +0.01%
===========================================
Files 2140 2141 +1
Lines 397273 397416 +143
===========================================
+ Hits 351891 352061 +170
+ Misses 45382 45355 -27
... and 26 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
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 for sage.manifolds and sage.tensor parts.
Thank you! |
|
📚 Description
Adding tags that make it possible/meaningful to doctest
sage.manifolds
,sage.tensor
even when some "standard" packages (pplpy
, the modularized distribution packaging providingsage.symbolic
andsage.manifolds
) are not installed. Also taking care ofsage.geometry.riemannian_manifolds
.# optional
doctest tags #34998📝 Checklist
⌛ Dependencies
This is just two commits on top of:
sage.features
: Addsage.libs.singular
, features for standard Python packages #35237