Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sage.geometry.hyperbolic_space: __matmul__ for composition of isometries #32212

Open
mkoeppe opened this issue Jul 16, 2021 · 13 comments
Open

sage.geometry.hyperbolic_space: __matmul__ for composition of isometries #32212

mkoeppe opened this issue Jul 16, 2021 · 13 comments

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Jul 16, 2021

Follow-up from #30244

Depends on #30244

CC: @tscrim

Component: geometry

Author: Matthias Koeppe

Branch/Commit: u/mkoeppe/sage_geometry_hyperbolic_space__use_actions_instead_of_overriding___mul______rmul__ @ 32c7d40

Issue created by migration from https://trac.sagemath.org/ticket/32212

@mkoeppe mkoeppe added this to the sage-9.4 milestone Jul 16, 2021
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jul 16, 2021

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jul 16, 2021

Dependencies: #30244

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jul 16, 2021

New commits:

1bb878cHyperbolicIsometry: Split out _composition from `__mul__` so that also @ between isometries works
923ebb7HyperbolicIsometryPD: Override _composition, not __mul__

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jul 16, 2021

Commit: 923ebb7

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 16, 2021

Changed commit from 923ebb7 to 32c7d40

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jul 16, 2021

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

66206f9FanMorphism: Define `__matmul__`, not __mul__
3006833FiniteSetEndoMap*: Define `__matmul__`, delegate to it from __mul__
de14c34TensorWithIndices: Make `__matmul__` an alias of __mul__
451eb95Map.__mul__: Add doctest output
348f680TensorWithIndices: Update doctests
e808163Merge #30244
305029aCoercionModel.verify_coercion_maps: Use @ instead of * for composition
1a6671fPrecomposedAction.__init__: Use @ instead of * for composition
2d8698fHyperbolicGeodesic*: More @ instead of *
32c7d40HyperbolicIsometry.__matmul__: Repair composition of isometries from different models

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jul 16, 2021

Author: Matthias Koeppe

@mkoeppe

This comment has been minimized.

@mkoeppe mkoeppe changed the title sage.geometry.hyperbolic_space: Use actions instead of overriding __mul__, __rmul__ sage.geometry.hyperbolic_space: __matmul__ for composition of isometries Jul 16, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.4, sage-9.5 Aug 9, 2021
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Dec 18, 2021

comment:6

Stalled in needs_review or needs_info; likely won't make it into Sage 9.5.

@mkoeppe mkoeppe modified the milestones: sage-9.5, sage-9.6 Dec 18, 2021
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 3, 2022

comment:7

needs rebase

@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 Apr 2, 2022
@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Aug 31, 2022
@mkoeppe mkoeppe removed this from the sage-9.8 milestone Jan 29, 2023
@grhkm21
Copy link
Contributor

grhkm21 commented Feb 12, 2024

@mkoeppe What should I do if I want to unblock this ticket? Review it here or {make a new branch on my fork with the commits, rebase, then PR}? (I assume "rebase" means just git merge and fixing conflicts right?)

Also I realised you are the author, but I can help with rebasing.

@grhkm21 grhkm21 self-assigned this Feb 12, 2024
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Feb 12, 2024

Yes, please feel free to take over. Preparing a PR is the best way forward.
Whether you use git merge or git rebase to refresh the branch is up to you

@grhkm21
Copy link
Contributor

grhkm21 commented Feb 12, 2024

Will do 👍🏻

I didn’t know git rebase is a thing, oops.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants