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: |
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
comment:2 Changed 12 months ago by
- Milestone changed from sage-9.4 to sage-9.5
comment:3 Changed 8 months ago by
- Milestone changed from sage-9.5 to sage-9.6
comment:4 Changed 4 months ago by
- Milestone changed from sage-9.6 to sage-9.7
Note: See
TracTickets for help on using
tickets.
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 thanfrom sage.manifolds.differentiable.operators import curl
.