Opened 22 months ago

Last modified 16 months ago

#30274 closed enhancement

Immutability of Tensor Fields — at Version 7

Reported by: gh-mjungmath Owned by:
Priority: major Milestone: sage-9.2
Component: manifolds Keywords: immutability
Cc: egourgoulhon, tscrim, mkoeppe Merged in:
Authors: Michael Jung Reviewers:
Report Upstream: N/A Work issues:
Branch: u/gh-mjungmath/immutability_of_tensor_fields (Commits, GitHub, GitLab) Commit: 0ba933a0b0aa4734e979f0e22b6d458790ff0d49
Dependencies: #30181 Stopgaps:

Status badges

Description (last modified by gh-mjungmath)

See #30261.

The immutability behavior for tensor fields appears a little bit tricky.

Change History (7)

comment:1 Changed 22 months ago by gh-mjungmath

  • Cc egourgoulhon tscrim mkoeppe added
  • Component changed from PLEASE CHANGE to manifolds
  • Description modified (diff)
  • Keywords immutability added
  • Type changed from PLEASE CHANGE to enhancement

comment:2 Changed 22 months ago by gh-mjungmath

  • Branch set to u/gh-mjungmath/immutability_of_tensor_fields

comment:3 Changed 22 months ago by gh-mjungmath

  • Commit set to 0ba933a0b0aa4734e979f0e22b6d458790ff0d49
  • Status changed from new to needs_review

New commits:

286d3c6FreeModuleTensor, Vector: Subclass a new class ModuleElementWithMutability
9df3d22sage.tensor.modules: Make all zero() and one() elements immutable
4373ea2FreeModuleTensor: Replace special casing of the immutable zero by is_immutable
a2ee8beModuleElementWithMutability.is_immutable, is_mutable: Change def to cpdef
fa6bd69Trac #30274: Merge branch 't/30181/immutable_elements_of_freemoduletensor' into t/30274/immutability_of_tensor_fields
0ba933aTrac #30274: first trial of immutability

comment:4 Changed 22 months ago by gh-mjungmath

  • Authors set to Michael Jung

comment:5 Changed 22 months ago by gh-mjungmath

Let's wait for the patchbot.

I had to make some modifications for automorphism fields. My doctests all passed.

comment:6 Changed 22 months ago by gh-mjungmath

  • Dependencies set to #30181

comment:7 Changed 22 months ago by gh-mjungmath

  • Description modified (diff)
Note: See TracTickets for help on using tickets.