Add satellite, cable, and whitehead_double functions - #74
Conversation
|
@alexeldridge Many thanks for doing this. Overall the code looks good, but I have two requests before merging:
Let me know if anything is unclear. |
|
I've added the methods to One thing I noticed while testing some edge cases: the |
1. Shortened lines. 2. Prefered string literal indicator `"` -> `'`. 3. Translate Markdown to re-Structured text as this needed for the docs.
work without an explicit import. Also hack around the need to do "from . import BraidTangle, IdentityBraid" as that will not work for the end-user.
|
@alexeldridge Thanks for the changes. I made few changes to the doctests to be compatible the auto-generator for the documentation for SnapPy, which uses reStructuredText rather than Markdown. Concerning I'll merge this PR shortly. |
|
Ok, merged and I put a note in SnapPy docs to mention this in the new features list for the next release. Thanks for contributing this! |
This change adds several functions to perform satellite operations on knots. Given an (n, n)-
Tanglerepresenting a pattern knot in the solid torus, thesatellitefunction returns the satellite of a given knot by that pattern optionally with a specified number of full twists. The satellite inherits its orientation from the pattern. Specialized functions for cables and Whitehead doubles are also implemented.