Opened 6 months ago

Last modified 3 months ago

#31766 new enhancement

ManifoldSubset: Replace use of _top_subsets by a generator

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-9.5
Component: manifolds Keywords:
Cc: gh-mjungmath, egourgoulhon 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 (4)

comment:1 Changed 6 months ago by mkoeppe

  • Dependencies changed from #31736 to #31736, #31764

comment:2 Changed 6 months ago by mkoeppe

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 6 months ago by gh-mjungmath

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 3 months ago by mkoeppe

  • Milestone changed from sage-9.4 to sage-9.5
Note: See TracTickets for help on using tickets.