Opened 10 months ago

Last modified 10 months ago

#32712 closed enhancement

sage.tensor.modules: make doctests involving SR optional — at Version 6

Reported by: egourgoulhon Owned by:
Priority: major Milestone: sage-9.5
Component: refactoring Keywords:
Cc: mkoeppe, tscrim Merged in:
Authors: Eric Gourgoulhon Reviewers:
Report Upstream: N/A Work issues:
Branch: public/manifolds/SR_doctests_optional_in_tensor_modules-32712 (Commits, GitHub, GitLab) Commit: c3ff5143e5d7f5b617a7678c43f4b23e7deaed06
Dependencies: #32708, #32614 Stopgaps:

Status badges

Description (last modified by egourgoulhon)

After #32415 and #32708, the code in the module sage.tensor.modules does no longer contain any dependency on symbolic parts of Sage. There remains however some doctests that invokes SR. They have been introduced to check a bug fix performed in #22520. In view of Sage modularization, they are marked # optional - sage.symbolic in this ticket.

Change History (6)

comment:1 Changed 10 months ago by egourgoulhon

  • Authors set to Eric Gourgoulhon
  • Branch set to public/manifolds/remove_SR_doctests_in_tensor_modules-32712
  • Cc mkoeppe tscrim added
  • Commit set to f46cce7058f1d5f26d1156f1b7c3100ba3660447
  • Status changed from new to needs_review

New commits:

1635bd3Remove import of Chart in sage.tensor.modules (#32708)
f46cce7Remove SR doctests in sage.tensor.modules (#32712)

comment:2 follow-up: Changed 10 months ago by mkoeppe

Instead of removing them, they could be marked as optional using #32614: See #32652 for an example

comment:3 in reply to: ↑ 2 Changed 10 months ago by egourgoulhon

Replying to mkoeppe:

Instead of removing them, they could be marked as optional using #32614: See #32652 for an example

Very good. Should the tag be # optional - sage.symbolic.ring ?

comment:4 follow-up: Changed 10 months ago by mkoeppe

  • Dependencies changed from #32708 to #32708, #32614

see src/sage/features/sagemath.py (after merging #32614): just sage.symbolic

comment:5 in reply to: ↑ 4 Changed 10 months ago by egourgoulhon

Replying to mkoeppe:

see src/sage/features/sagemath.py (after merging #32614): just sage.symbolic

OK, perfect.

comment:6 Changed 10 months ago by egourgoulhon

  • Branch changed from public/manifolds/remove_SR_doctests_in_tensor_modules-32712 to public/manifolds/SR_doctests_optional_in_tensor_modules-32712
  • Commit changed from f46cce7058f1d5f26d1156f1b7c3100ba3660447 to c3ff5143e5d7f5b617a7678c43f4b23e7deaed06
  • Description modified (diff)
  • Summary changed from sage.tensor.modules: remove doctests involving the symbolic ring to sage.tensor.modules: make doctests involving SR optional

New commits:

c3ff514Make SR doctests optional in sage.tensor.modules (#32712)
Note: See TracTickets for help on using tickets.