Modularization of sagelib: Break out a separate package sagemath-standard-no-symbolics#35150
Closed
mkoeppe wants to merge 860 commits intosagemath:developfrom
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #35150 +/- ##
===========================================
- Coverage 88.62% 87.73% -0.89%
===========================================
Files 2148 2147 -1
Lines 398653 398817 +164
===========================================
- Hits 353308 349908 -3400
- Misses 45345 48909 +3564
... and 264 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. |
4 tasks
Collaborator
|
This is the last part of the log of testing this distribution ...
sage -t --random-seed=228944954937657511694224272655345237546 /Users/kwankyu/GitHub/sage-dev/local/share/doc/sage/html/en/thematic_tutorials/_sources/geometry/visualization.rst.txt # 1 doctest failed
----------------------------------------------------------------------
Total time for all tests: 1376.9 seconds
cpu time: 6674.4 seconds
cumulative wall time: 8638.3 seconds
Features detected for doctesting: 4ti2,csdp,cvxopt,dvipng,gfan,graphviz,imagemagick,nauty,palp,pandoc,pdf2svg,pdftocairo,polytopes_db,sage.combinat,sage.geometry.polyhedron,sage.graphs,sage.groups,sage.libs.pari,sage.misc.cython,sage.plot,sage.rings.number_field,sage.rings.padics,sage.rings.real_double,sage.rings.real_mpfr,sagemath_doc_html,sphinx
pytest is not installed in the venv, skip checking tests that rely on it
sagepython-sagewheels-nopypi: 2028389 C exit 5 (1388.49 seconds) /Users/kwankyu/GitHub/sage-dev/pkgs/sagemath-standard-no-symbolics> sage -t -p --initial --installed pid=81355 [tox/execute/api.py:275]
.pkg-sagepython-sagewheels-nopypi: 2028398 W _exit> python /usr/local/Cellar/tox/4.4.6/libexec/lib/python3.11/site-packages/pyproject_api/_backend.py True setuptools.build_meta [tox/tox_env/api.py:428]
Backend: run command _exit with args {}
Backend: Wrote response {'return': 0} to /var/folders/td/fw1q9ljs311ggyph77rs53_40000gn/T/pep517__exit-jz2f044r.json
.pkg-sagepython-sagewheels-nopypi: 2028446 I exit None (0.05 seconds) /Users/kwankyu/GitHub/sage-dev/pkgs/sagemath-standard-no-symbolics> python /usr/local/Cellar/tox/4.4.6/libexec/lib/python3.11/site-packages/pyproject_api/_backend.py True setuptools.build_meta pid=78662 [tox/execute/api.py:275]
.pkg-sagepython-sagewheels-nopypi: 2028566 D delete package /Users/kwankyu/GitHub/sage-dev/pkgs/sagemath-standard-no-symbolics/.tox/.tmp/package/1/sagemath-standard-no-symbolics-10.0b2.tar.gz [tox/tox_env/python/virtual_env/package/pyproject.py:179]
sagepython-sagewheels-nopypi: FAIL code 5 (2028.33=setup[636.16]+cmd[1.73,1.95,1388.49] seconds)
evaluation failed :( (2028.45 seconds)I don't understand but the last lines seem to say something failed. |
Contributor
Author
|
Yes, this distribution still has lots and lots of doctest failures, so |
added 26 commits
March 18, 2023 14:13
…e.libs.pari sage.rings.padics
…tions, sage.symbolic into method
…ions into methods
…orts into methods
…ting some methods
…orms_modularization
…padics}; doctest cosmetics
… to sage.combinat.subset (with deprecation)
…to 10.0.rc1-merged
…n' into 10.0.rc1-merged
SageMath version 10.0.rc3, Release Date: 2023-05-12
…__break_out_a_separate_package_sagemath_standard_no_symbolics
|
Documentation preview for this PR is ready! 🎉 |
Contributor
Author
|
Merged this branch into #35095 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📚 Description
Fixes #32601
📝 Checklist
⌛ Dependencies
# optionaldoctest tags #34998