Opened 2 years ago

Last modified 2 years ago

#28628 closed defect

Tensor Fields: set_restriction Behaviour — at Version 2

Reported by: gh-DeRhamSource Owned by:
Priority: major Milestone: sage-9.0
Component: geometry Keywords: manifolds, tensor
Cc: egourgoulhon Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: u/gh-DeRhamSource/tensor_fields__set_restriction_behaviour (Commits, GitHub, GitLab) Commit: e30ec5b88f8c3144c08e731d85c4c79e58595cc4
Dependencies: Stopgaps:

Status badges

Description (last modified by gh-DeRhamSource)

The following code leads to an error:

sage: M = Manifold(2, 'M')
sage: X.<x,y> = M.chart()
sage: eX = X.frame()
sage: a = M.one_form()
sage: b = M.one_form()
sage: a[eX,:] = x*y, x+y
sage: b.set_restriction(a)
sage: b.display()

ValueError   Traceback (most recent call last)
...

ValueError: no basis could be found for computing the components in the Coordinate frame (M, (d/dx,d/dy))

Mathematically, this should not happen.

A method copy_from is added, and in case of the same domain, the copy_from method is invoked, when set_restriction is used.

Change History (2)

comment:1 Changed 2 years ago by gh-DeRhamSource

  • Branch set to u/gh-DeRhamSource/tensor_fields__set_restriction_behaviour

comment:2 Changed 2 years ago by gh-DeRhamSource

  • Cc egourgoulhon added
  • Commit set to e30ec5b88f8c3144c08e731d85c4c79e58595cc4
  • Description modified (diff)
  • Keywords manifolds added
  • Status changed from new to needs_review
  • Type changed from PLEASE CHANGE to defect

New commits:

e30ec5b'set_restriction' modified and 'copy_from' method added
Note: See TracTickets for help on using tickets.