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

symplectic basis for triangulated 3 manifolds #117

Merged
merged 1 commit into from
Oct 13, 2023

Conversation

jchilds0
Copy link
Contributor

@jchilds0 jchilds0 commented Oct 3, 2023

An algorithm for constructing oscillating curves on triangulated 3-manifolds with tori boundary components. These oscillating curves are used to extend the Neumann Zagier matrix to one which is symplectic up to factors of 2.

See https://arxiv.org/abs/2208.06969

You can test the algorithm on random link exteriors by running dev/symplectic_basis/test_base.py. There's also a cmake file in that directory if you want to compile and run just the C portion directly with symplectic_basis_test.c or symplectic_basis_main.c.

Tested on the knots in the extended HTLinkExterior census (312k knots), and with Valgrind memcheck on symplectic_basis_test.c. Current implementation only works on manifolds with one cusp, its not clear if oscillating curves exist on manifolds with more than one cusp. The code has everything to run on manifolds with multiple cusps, except finding the paths for oscillating curves which run through multiple cusps.

@jchilds0
Copy link
Contributor Author

jchilds0 commented Oct 3, 2023

It looks like the windows wheels failed from a timeout when downloading the venv package, can we try a re run?

@NathanDunfield
Copy link
Member

It looks like the windows wheels failed from a timeout when downloading the venv package, can we try a re run?

Sure, I triggered the re-run and everything is now green. We'll have a look at the PR in detail shortly.

@NathanDunfield NathanDunfield merged commit 4df00b4 into 3-manifolds:master Oct 13, 2023
14 checks passed
@NathanDunfield
Copy link
Member

We've merged this PR, thanks for a very well-done contribution!

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

Successfully merging this pull request may close these issues.

2 participants