Opened 4 years ago

Closed 4 years ago

#26012 closed defect (fixed)

Fix bug concerning restrictions of tensorfields

Reported by: gh-FlorentinJ Owned by:
Priority: major Milestone: sage-8.4
Component: geometry Keywords: restriction, extension, tensorfield, manifold
Cc: Eric Gourgoulhon Merged in:
Authors: Florentin Jaffredo Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: 043f084 (Commits, GitHub, GitLab) Commit: 043f08436dd36cd853d47a3855a260afd19420b5
Dependencies: Stopgaps:

Status badges


The attributes _restrictions_graph and _extensions_graph, introduced in #25417 were not affected by the method _del_derived, whereas _restrictions was, causing some issues.

Change History (5)

comment:1 Changed 4 years ago by gh-FlorentinJ

Status: newneeds_review

comment:2 Changed 4 years ago by Travis Scrimshaw

Do you have an explicit example where something was going wrong? If so, could you add it as a test to some TESTS:: block with a message saying something like ``_restrictions_graph`` and ``_extensions_graph`` was not changed (see :trac:`26012`)::

comment:3 Changed 4 years ago by git

Commit: e5787adef1d5c47f63190f4c478b0414cae7c389043f08436dd36cd853d47a3855a260afd19420b5

Branch pushed to git repo; I updated commit sha1. New commits:

043f084Added TESTS:: block, typo

comment:4 Changed 4 years ago by Travis Scrimshaw

Reviewers: Travis Scrimshaw
Status: needs_reviewpositive_review

Thanks. LGTM.

comment:5 Changed 4 years ago by Volker Braun

Branch: public/manifolds/better_restrictions_bugfix043f08436dd36cd853d47a3855a260afd19420b5
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.