Opened 13 months ago

Last modified 4 months ago

#32274 new enhancement

Restructure manifold module

Reported by: gh-mjungmath Owned by:
Priority: major Milestone: sage-9.7
Component: manifolds Keywords:
Cc: egourgoulhon, tscrim, mkoeppe, gh-tobiasdiez, nthiery Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

I propose to restructure the manifold module. For example:

  • separate the code related to the (pseudo-)Riemannian setting and dedicate a own module to it
  • create a new module for vector bundles

Some things are currently very vague:

  • operators.py clearly contains code related to differentiable manifolds but stays in the main module
  • we have a submodule subsets but the separation is currently not very consistent; subset.py for example still belongs to the main module

Change History (4)

comment:1 Changed 12 months ago by egourgoulhon

OK for this, provided there is some deprecation period (last time there were such changes, it broke codes relying on Sage sources, like the kerrgeodesic_gw package).

Regarding operators.py, I agree with your remark; however, I would keep it in the main module for convenience of the end user, from sage.manifolds.operators import curl being shorter than from sage.manifolds.differentiable.operators import curl.

comment:2 Changed 12 months ago by mkoeppe

  • Milestone changed from sage-9.4 to sage-9.5

comment:3 Changed 8 months ago by mkoeppe

  • Milestone changed from sage-9.5 to sage-9.6

comment:4 Changed 4 months ago by mkoeppe

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