-
Notifications
You must be signed in to change notification settings - Fork 10
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
Implement Iso Delaunay Tessellation #163
base: master
Are you sure you want to change the base?
Conversation
Documentation preview for this PR is ready! 🎉 |
We are working with a student (Oscar) on using iso-Delaunay and spine to list more efficiently saddle connections. Right now, the code in the branch does not really makes the distinction between
|
That sounds like an interesting use case for the iso-Delaunay tessellation! Making IsoDelaunayTessellation the former would be consistent with Bowman’s paper.
Are there still plans to implement the Veech group element discovery portion of the code?
… On Apr 25, 2024, at 9:07 AM, Vincent Delecroix ***@***.***> wrote:
We are working with a student (Oscar) on using iso-Delaunay and spine to list more efficiently saddle connections. Right now, the code in the branch does not really makes the distinction between
the tessellation of $\mathbb{H}^2$ which always have infinitely many regions
the tessellation of $\mathbb{H}^2 / \Gamma$
To me the IsoDelaunayTessellation should be the first of the two. The fact that it could be used to discover elements in the Veech group is definitely an important feature. But to my mind it would be better not to force such check for each face exploration.
—
Reply to this email directly, view it on GitHub <#163 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQGU3UTRQAFJS5E3YGIXJO3Y7D5YJAVCNFSM5UPKMIDKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMBXG4YTGOBVGI3Q>.
You are receiving this because you were mentioned.
|
Yes. This is another important motivation! But for that purpose I would implement a different |
Also, it feels like the |
fixes #160