Opened 17 months ago

Last modified 5 weeks ago

#31766 new enhancement

ManifoldSubset: Replace use of _top_subsets by a generator

Reported by: Matthias Köppe Owned by:
Priority: major Milestone: sage-9.8
Component: manifolds Keywords:
Cc: Michael Jung, Eric Gourgoulhon Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #31736, #31764 Stopgaps:

Status badges

Description

... of the maximal elements of the subset poset.

Change History (7)

comment:1 Changed 17 months ago by Matthias Köppe

Dependencies: #31736#31736, #31764

comment:2 Changed 17 months ago by Matthias Köppe

Actually _top_subsets is only used in one place, in TensorField.along.

Perhaps it is possible to get rid of it altogether?

comment:3 Changed 17 months ago by Michael Jung

Oh, something similar happens for the volume form and the display method of scalar fields. Moreover, see _get_min_covering in manifold.py.

comment:4 Changed 15 months ago by Matthias Köppe

Milestone: sage-9.4sage-9.5

comment:5 Changed 10 months ago by Matthias Köppe

Milestone: sage-9.5sage-9.6

comment:6 Changed 7 months ago by Matthias Köppe

Milestone: sage-9.6sage-9.7

comment:7 Changed 5 weeks ago by Matthias Köppe

Milestone: sage-9.7sage-9.8
Note: See TracTickets for help on using tickets.