#32116 closed enhancement (fixed)

Chart._parse_coordinates

Reported by: Matthias Köppe Owned by:
Priority: major Milestone: sage-9.4
Component: manifolds Keywords:
Cc: Eric Gourgoulhon, Volker Braun Merged in:
Authors: Matthias Koeppe Reviewers: Eric Gourgoulhon
Report Upstream: N/A Work issues:
Branch: u/mkoeppe/chart__parse_coordinates (Commits, GitHub, GitLab) Commit: 3d1c44d7e5dde0c2a65740a615998205b48be075
Dependencies: #32009 Stopgaps:

Status badges

Description

Replacing _init_coordinates.

Split out from #31894, for #32102.

Change History (13)

comment:1 Changed 17 months ago by Matthias Köppe

Dependencies: #32009

comment:2 Changed 17 months ago by Matthias Köppe

Branch: u/mkoeppe/chart__parse_coordinates

comment:3 Changed 17 months ago by Matthias Köppe

Authors: Matthias Koeppe
Commit: 4db499543d24006d4ea3b87c956baa5f0bee03a7
Status: newneeds_review

New commits:

8ba174cEliminate direct use of Chart._domain
21297f3Merge #32009
deace83Chart: Replace _init_coordinates by _parse_coordinates
4db4995Chart: Fix up __classcall__ and _parse_coordinates by avoiding unhashable things

comment:4 Changed 17 months ago by git

Commit: 4db499543d24006d4ea3b87c956baa5f0bee03a7fc59c9d6d46faeefd12626ab9946631de2608424

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

fc59c9dChart.__classcall__: Add doctest

comment:5 Changed 17 months ago by git

Commit: fc59c9d6d46faeefd12626ab9946631de26084241742fdccd1e491fa6c15b18acf9990178a8495c0

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

f43b358Chart._init_coordinates: Fix up use of domain
4e316e9Fix bug in Chart.__init__ regarding the determination of top charts (Trac #32112)
907c9bcMerge #32112
1742fdcMerge #32009

comment:6 Changed 17 months ago by Eric Gourgoulhon

In line 413 of chart.py, the doctest

sage: Chart._parse_coordinates(M, ['z1:\zeta_1', r'z2:\zeta_2'])

fails due to a missing raw string marker.

comment:7 Changed 17 months ago by Eric Gourgoulhon

Apart from comment:6, this looks nice!

Last edited 17 months ago by Eric Gourgoulhon (previous) (diff)

comment:8 Changed 17 months ago by git

Commit: 1742fdccd1e491fa6c15b18acf9990178a8495c09ac183438b80e55d6d994d61f5d92a7c5e056026

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

9ac1834src/sage/manifolds/chart.py: Add raw string marker

comment:9 Changed 17 months ago by Eric Gourgoulhon

Reviewers: Eric Gourgoulhon
Status: needs_reviewpositive_review

Thanks!

comment:10 Changed 17 months ago by Matthias Köppe

Thank you!

comment:11 Changed 17 months ago by git

Commit: 9ac183438b80e55d6d994d61f5d92a7c5e0560263d1c44d7e5dde0c2a65740a615998205b48be075
Status: positive_reviewneeds_review

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. Last 10 new commits:

69d045aConditionSet: In doctests, do not rename ZZ^2 etc.
daeb91esrc/sage/sets/set.py: Fix docstring markup
2cf2199Merge #32015
1eb270asrc/sage/docs/conf.py: Add more \ensuremath to \DeclareUnicodeCharacter
2682469src/sage/interfaces/sympy_wrapper.py: Use Family, not Set, in doctests to make sure that the SageSet wrapper is actually used
753babbSet_object_enumerated._sympy_: Translate empty sets to EmptySet
141ecdeMerge #32015
bf62543Merge branch 't/32089/conditionset__conditionset_callable_symbolic_expression' into t/32009/eliminate_direct_use_of_the_chart__domain_attribute
ea08261Merge #32009
3d1c44dMerge tag '9.4.beta5' into t/32116/chart__parse_coordinates

comment:12 Changed 17 months ago by Matthias Köppe

Status: needs_reviewpositive_review

comment:13 Changed 16 months ago by Matthias Köppe

Cc: Volker Braun added
Resolution: fixed
Status: positive_reviewclosed

Apparently this has been merged as part of #32089.

Note: See TracTickets for help on using tickets.