sage.tensor.modules: remove dependency on sage.manifolds
Closed this issue · 6 comments
egourgoulhon commented
In view of Sage modularization, this ticket removes the only import from sage.manifolds in the module sage.tensor.modules.
This was an import of Chart in FreeModuleTensor._preparse_display. This is avoided by reimplementing _preparse_display in TensorFieldParal.
Component: manifolds
Keywords: tensor display
Author: Eric Gourgoulhon
Branch/Commit: 1635bd3
Reviewer: Matthias Koeppe
Issue created by migration from https://trac.sagemath.org/ticket/32708
egourgoulhon commented
Commit: 1635bd3
egourgoulhon commented
New commits:
1635bd3 | Remove import of Chart in sage.tensor.modules (#32708) |
egourgoulhon commented
mkoeppe commented
Reviewer: Matthias Koeppe
egourgoulhon commented
comment:3
Thank you for the review!
vbraun commented
Changed branch from public/manifolds/no_chart_import_in_tensor_modules-32708 to 1635bd3