Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

Commit

Permalink
Trac #32415: Remove unnecessary uses of symbolic functions in sage.te…
Browse files Browse the repository at this point in the history
…nsor.modules

Similar to what is done in #32411.

This is part of Meta-ticket #29705 (modularization) - to remove
unnecessary dependencies on the distribution `sagemath-symbolics`
(#31695)

URL: https://trac.sagemath.org/32415
Reported by: mkoeppe
Ticket author(s): Matthias Koeppe
Reviewer(s): Eric Gourgoulhon
  • Loading branch information
Release Manager committed Sep 13, 2021
2 parents f91cda2 + ac82dd2 commit a2af3f6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/sage/tensor/modules/comp.py
Original file line number Diff line number Diff line change
Expand Up @@ -5446,7 +5446,7 @@ def interior_product(self, other):
True
"""
from sage.functions.other import factorial
from sage.arith.all import factorial
# Sanity checks:
if not isinstance(other, CompFullyAntiSym):
raise TypeError("{} is not a fully antisymmetric ".format(other) +
Expand Down
4 changes: 2 additions & 2 deletions src/sage/tensor/modules/ext_pow_free_module.py
Original file line number Diff line number Diff line change
Expand Up @@ -236,7 +236,7 @@ def __init__(self, fmodule, degree, name=None, latex_name=None):
sage: TestSuite(A).run()
"""
from sage.functions.other import binomial
from sage.arith.all import binomial
from sage.typeset.unicode_characters import unicode_bigwedge
self._fmodule = fmodule
self._degree = ZZ(degree)
Expand Down Expand Up @@ -631,7 +631,7 @@ def __init__(self, fmodule, degree, name=None, latex_name=None):
sage: TestSuite(A).run()
"""
from sage.functions.other import binomial
from sage.arith.all import binomial
from sage.typeset.unicode_characters import unicode_bigwedge
self._fmodule = fmodule
self._degree = ZZ(degree)
Expand Down

0 comments on commit a2af3f6

Please sign in to comment.