Opened 4 years ago

Closed 4 years ago

#24396 closed defect (fixed)

test failure in manifolds/differentiable/tensorfield.py (8.2.beta0)

Reported by: vdelecroix Owned by:
Priority: blocker Milestone: sage-8.2
Component: documentation Keywords:
Cc: ​egourgoulhon Merged in:
Authors: Vincent Delecroix Reviewers: Eric Gourgoulhon
Report Upstream: N/A Work issues:
Branch: 95bd140 (Commits, GitHub, GitLab) Commit: 95bd140bddeb7bdc0f0fd2656206d41c8106429c
Dependencies: Stopgaps:

Status badges

Description (last modified by vdelecroix)

The following fails when --long is not specified

$ sage -t manifolds/differentiable/tensorfield.py
**********************************************************************
File "manifolds/differentiable/tensorfield.py", line 295, in sage.manifolds.differentiable.tensorfield.TensorField
Failed example:
    t[eV,1,1,c_uv].expr()
Exception raised:
    Traceback (most recent call last):
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 517, in _run
        self.compile_and_execute(example, compiler, test.globs)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 920, in compile_and_execute
        exec(compiled, globs)
      File "<doctest sage.manifolds.differentiable.tensorfield.TensorField[36]>", line 1, in <module>
        t[eV,Integer(1),Integer(1),c_uv].expr()
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/manifolds/differentiable/tensorfield.py", line 1472, in __getitem__
        return self.comp(frame)[args]
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/manifolds/differentiable/tensorfield.py", line 1235, in comp
        return rst.comp(basis=basis, from_basis=from_basis)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/manifolds/differentiable/tensorfield_paral.py", line 921, in comp
        from_basis=from_basis)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/tensor/modules/free_module_tensor.py", line 1076, in components
        "the components in the {}".format(basis))
    ValueError: no basis could be found for computing the components in the Coordinate frame (V, (d/du,d/dv))
**********************************************************************
File "manifolds/differentiable/tensorfield.py", line 298, in sage.manifolds.differentiable.tensorfield.TensorField
Failed example:
    type(t[eV,1,1,c_uv].expr())
Exception raised:
    Traceback (most recent call last):
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 517, in _run
        self.compile_and_execute(example, compiler, test.globs)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 920, in compile_and_execute
        exec(compiled, globs)
      File "<doctest sage.manifolds.differentiable.tensorfield.TensorField[37]>", line 1, in <module>
        type(t[eV,Integer(1),Integer(1),c_uv].expr())
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/manifolds/differentiable/tensorfield.py", line 1472, in __getitem__
        return self.comp(frame)[args]
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/manifolds/differentiable/tensorfield.py", line 1235, in comp
        return rst.comp(basis=basis, from_basis=from_basis)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/manifolds/differentiable/tensorfield_paral.py", line 921, in comp
        from_basis=from_basis)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/tensor/modules/free_module_tensor.py", line 1076, in components
        "the components in the {}".format(basis))
    ValueError: no basis could be found for computing the components in the Coordinate frame (V, (d/du,d/dv))
**********************************************************************
File "manifolds/differentiable/tensorfield.py", line 318, in sage.manifolds.differentiable.tensorfield.TensorField
Failed example:
    f = t(a,b)
Exception raised:
    Traceback (most recent call last):
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 517, in _run
        self.compile_and_execute(example, compiler, test.globs)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 920, in compile_and_execute
        exec(compiled, globs)
      File "<doctest sage.manifolds.differentiable.tensorfield.TensorField[46]>", line 1, in <module>
        f = t(a,b)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/manifolds/differentiable/tensorfield.py", line 2348, in __call__
        resu_rr = self_rr(*args_rr)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/manifolds/differentiable/tensorfield_paral.py", line 1461, in __call__
        return FreeModuleTensor.__call__(self_r, *args_r)
      File "/home/vdelecro/sage_patchbot/local/lib/python2.7/site-packages/sage/tensor/modules/free_module_tensor.py", line 2130, in __call__
        raise ValueError("no common basis for the components")
    ValueError: no common basis for the components
Killing test manifolds/differentiable/tensorfield.py
----------------------------------------------------------------------
Doctests interrupted: 0/1 files tested
----------------------------------------------------------------------
Total time for all tests: 26.5 seconds
    cpu time: 0.0 seconds
    cumulative wall time: 0.0 seconds

Change History (4)

comment:1 Changed 4 years ago by vdelecroix

  • Component changed from geometry to documentation
  • Description modified (diff)
  • Priority changed from major to blocker

comment:2 Changed 4 years ago by vdelecroix

  • Authors set to Vincent Delecroix
  • Branch set to u/vdelecroix/24396
  • Commit set to 95bd140bddeb7bdc0f0fd2656206d41c8106429c
  • Status changed from new to needs_review

New commits:

95bd14024396: fix doctest failure in manifolds

comment:3 Changed 4 years ago by egourgoulhon

  • Reviewers set to Eric Gourgoulhon
  • Status changed from needs_review to positive_review

Thank you for the fix!

comment:4 Changed 4 years ago by vbraun

  • Branch changed from u/vdelecroix/24396 to 95bd140bddeb7bdc0f0fd2656206d41c8106429c
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.