Opened 3 months ago

Last modified 2 months ago

#32274 new enhancement

Restructure manifold module

Reported by: gh-mjungmath Owned by:
Priority: major Milestone: sage-9.5
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 (2)

comment:1 Changed 2 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 2 months ago by mkoeppe

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