sage.modules, sage.matrix: Remove import-time dependencies on the symbolic ring

mkoeppe opened this issue 4 years ago · 8 comments

mkoeppe commented 4 years ago

(cherry-picked from #32432)

CC: @kliem @tscrim

Component: linear algebra

Author: Matthias Koeppe

Branch/Commit: acca6c2

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/32593

mkoeppe commented 4 years ago

Branch: u/mkoeppe/sage_modules__sage_matrix__remove_import_time_dependencies_on_the_symbolic_ring

mkoeppe commented 4 years ago

Commit: acca6c2

mkoeppe commented 4 years ago

Author: Matthias Koeppe

mkoeppe commented 4 years ago

New commits:

`1b0a5ab`	`src/sage/modules/vector_space_morphism.py: Do not fail if sage.symbol is not present`
`83b5038`	`src/sage/modules/free_module_element.pyx: Move import from sage.misc.derivative into method`
`acca6c2`	`src/sage/matrix/matrix2.pyx: Move import from sage.misc.derivative into method`

tscrim commented 4 years ago

Reviewer: Travis Scrimshaw

tscrim commented 4 years ago

comment:3

LGTM (assuming a green patchbot).

mkoeppe commented 4 years ago

comment:4

Thanks!

vbraun commented 4 years ago

Changed branch from u/mkoeppe/sage_modules__sage_matrix__remove_import_time_dependencies_on_the_symbolic_ring to acca6c2