-
-
Notifications
You must be signed in to change notification settings - Fork 654
Fix broken KnotInfo URL #40553
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
Fix broken KnotInfo URL #40553
Changes from 3 commits
5e34119
633223c
89179ce
6e1bacc
1c469d8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
tarball=database_knotinfo-VERSION.tar.gz | ||
sha1=d94335a4839946e73458064d6ad1def9595101c7 | ||
sha256=9065ec00e22bd1e43716b3e0c2d1bb9e2c32e504a24f11d81ab556428bb4aa7f | ||
sha1=5d9d19ead2d50e7ab6d48fddadb0a7635a9c7ca6 | ||
sha256=5c005386d6f3ffdca3310a1721e377f570272f702a6c81f7d7215fe3935baaf7 | ||
upstream_url=https://files.pythonhosted.org/packages/source/d/database_knotinfo/database_knotinfo-VERSION.tar.gz |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
2024.10.1 | ||
2025.8.7 |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,8 +3,8 @@ | |
|
||
This module contains the class :class:`KnotInfoDataBase` and auxiliary classes | ||
for it, which serves as an interface to the lists of named knots and links provided | ||
at the web-pages `KnotInfo <https://knotinfo.math.indiana.edu/>`__ and | ||
`LinkInfo <https://linkinfo.sitehost.iu.edu>`__. | ||
at the web-pages `KnotInfo <https://knotinfo.org/>`__ and | ||
`LinkInfo <https://https://link-info-repo.onrender.com/>`__. | ||
|
||
To use the database, you need to install the optional :ref:`database_knotinfo | ||
<spkg_database_knotinfo>` package by the Sage command :: | ||
|
@@ -48,8 +48,8 @@ | |
class KnotInfoColumnTypes(Enum): | ||
r""" | ||
Enum class to specify if a column from the table of knots and links provided | ||
at the web-pages `KnotInfo <https://knotinfo.math.indiana.edu/>`__ and | ||
`LinkInfo <https://linkinfo.sitehost.iu.edu>`__. is used for knots only, | ||
at the web-pages `KnotInfo <https://knotinfo.org/>`__ and | ||
`LinkInfo <https://https://link-info-repo.onrender.com/>`__. is used for knots only, | ||
links only or both. | ||
|
||
EXAMPLES:: | ||
|
@@ -68,8 +68,8 @@ class KnotInfoColumnTypes(Enum): | |
class KnotInfoColumns(Enum): | ||
r""" | ||
Enum class to select a column from the table of knots and links provided | ||
at the web-pages `KnotInfo <https://knotinfo.math.indiana.edu/>`__ and | ||
`LinkInfo <https://linkinfo.sitehost.iu.edu>`__. | ||
at the web-pages `KnotInfo <https://knotinfo.org/>`__ and | ||
`LinkInfo <https://https://link-info-repo.onrender.com/>`__. | ||
|
||
EXAMPLES:: | ||
|
||
|
@@ -199,7 +199,7 @@ def url(self): | |
sage: from sage.databases.knotinfo_db import KnotInfoDataBase | ||
sage: ki_db = KnotInfoDataBase() | ||
sage: ki_db.filename.knots.url() | ||
'https://knotinfo.math.indiana.edu/' | ||
'https://knotinfo.org/' | ||
""" | ||
if self == KnotInfoFilename.knots: | ||
return self.value[0] | ||
|
@@ -304,7 +304,7 @@ def description_url(self, column): | |
sage: from sage.databases.knotinfo_db import KnotInfoDataBase | ||
sage: ki_db = KnotInfoDataBase() | ||
sage: ki_db.filename.knots.description_url(ki_db.columns().braid_notation) | ||
'https://knotinfo.math.indiana.edu/descriptions/braid_notation.html' | ||
'https://knotinfo.org/descriptions/braid_notation.html' | ||
""" | ||
return '%sdescriptions/%s.html' % (self.url(), column.name) | ||
|
||
|
@@ -317,17 +317,17 @@ def diagram_url(self, fname, single=False): | |
sage: from sage.databases.knotinfo_db import KnotInfoDataBase | ||
sage: ki_db = KnotInfoDataBase() | ||
sage: ki_db.filename.knots.diagram_url('3_1-50.png') | ||
'https://knotinfo.math.indiana.edu/diagram_display.php?3_1-50.png' | ||
'https://knotinfo.org/diagram_display.php?3_1-50.png' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. When I try this link, I get a page with
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Interesting! I checked
which launches the correct site: |
||
sage: ki_db.filename.knots.diagram_url('3_1', single=True) | ||
'https://knotinfo.math.indiana.edu/diagrams/3_1' | ||
'https://knotinfo.org/diagrams/3_1' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I get
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. See the previous issue. |
||
""" | ||
if single: | ||
return '%sdiagrams/%s' % (self.url(), fname) | ||
else: | ||
return '%sdiagram_display.php?%s' % (self.url(), fname) | ||
|
||
knots = ['https://knotinfo.math.indiana.edu/', 'knotinfo_data_complete'] | ||
links = ['https://linkinfo.sitehost.iu.edu/', 'linkinfo_data_complete'] | ||
knots = ['https://knotinfo.org/', 'knotinfo_data_complete'] | ||
links = ['https://link-info-repo.onrender.com/', 'linkinfo_data_complete'] | ||
|
||
|
||
#---------------------------------------------------------------------------------------------------------------------------- | ||
|
@@ -347,7 +347,7 @@ class KnotInfoDataBase(SageObject, UniqueRepresentation): | |
sage: from sage.databases.knotinfo_db import KnotInfoDataBase | ||
sage: ki_db = KnotInfoDataBase() | ||
sage: ki_db.filename.knots | ||
<KnotInfoFilename.knots: ['https://knotinfo.math.indiana.edu/', | ||
<KnotInfoFilename.knots: ['https://knotinfo.org/', | ||
'knotinfo_data_complete']> | ||
""" | ||
|
||
|
@@ -362,7 +362,7 @@ def __init__(self, install=False): | |
sage: from sage.databases.knotinfo_db import KnotInfoDataBase | ||
sage: ki_db = KnotInfoDataBase() | ||
sage: ki_db.filename.links | ||
<KnotInfoFilename.links: ['https://linkinfo.sitehost.iu.edu/', | ||
<KnotInfoFilename.links: ['https://link-info-repo.onrender.com/', | ||
'linkinfo_data_complete']> | ||
""" | ||
# some constants | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,8 +4,8 @@ | |
|
||
This module contains the class :class:`KnotInfoBase` which is derived from | ||
:class:`Enum` and provides knots and links listed in the databases at the | ||
web-pages `KnotInfo <https://knotinfo.math.indiana.edu/>`__ | ||
and `LinkInfo <https://linkinfo.sitehost.iu.edu/>`__ as its items. | ||
web-pages `KnotInfo <https://knotinfo.org/>`__ | ||
and `LinkInfo <https://link-info-repo.onrender.com/>`__ as its items. | ||
|
||
This interface contains a set of about twenty knots and links statically as | ||
demonstration cases. The complete database can be installed as an optional Sage | ||
|
@@ -209,8 +209,8 @@ | |
|
||
REFERENCES: | ||
|
||
- `KnotInfo <https://knotinfo.math.indiana.edu/>`__ | ||
- `LinkInfo <https://linkinfo.sitehost.iu.edu/>`__ | ||
- `KnotInfo <https://knotinfo.org/>`__ | ||
- `LinkInfo <https://link-info-repo.onrender.com/>`__ | ||
|
||
|
||
AUTHORS: | ||
|
@@ -469,7 +469,7 @@ def is_minimal(self, link) -> bool: | |
class KnotInfoBase(Enum): | ||
r""" | ||
Enum class to select the knots and links listed in the databases at the web-pages | ||
`KnotInfo <https://knotinfo.math.indiana.edu/>`__ and `LinkInfo <https://linkinfo.sitehost.iu.edu/>`__. | ||
`KnotInfo <https://knotinfo.org/>`__ and `LinkInfo <https://link-info-repo.onrender.com/>`__. | ||
|
||
EXAMPLES:: | ||
|
||
|
@@ -1323,7 +1323,7 @@ def cosmetic_crossing_conjecture_verified(self): | |
Return whether the Cosmetic Crossing Conjecture has been verified | ||
for ``self``. | ||
|
||
From the KnotInfo `description page <https://knotinfo.math.indiana.edu/descriptions/cosmetic_crossing.html>`__: | ||
From the KnotInfo `description page <https://knotinfo.org/descriptions/cosmetic_crossing.html>`__: | ||
|
||
A crossing change in a diagram of a knot ``K`` is called cosmetic if | ||
the resulting diagram also represents ``K``. The cosmetic crossing | ||
|
@@ -1359,7 +1359,7 @@ def homfly_polynomial(self, var1='v', var2='z', original=False): | |
|
||
The HOMFLY-PT polynomial `P(L)` of a link `L` satisfies the following skein | ||
relation (see the corresponding `KnotInfo description page | ||
<https://knotinfo.math.indiana.edu/descriptions/jones_homfly_kauffman_description/polynomial_defn.html)>`__): | ||
<https://knotinfo.org/descriptions/jones_homfly_kauffman_description/polynomial_defn.html)>`__): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. there is a There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed! |
||
|
||
.. MATH:: | ||
|
||
|
@@ -1460,7 +1460,7 @@ def kauffman_polynomial(self, var1='a', var2='z', original=False): | |
under regular isotopy `\Delta (L) = a^{w(L)} F(L)` where `w(L)` is the | ||
writhe of the link `L` satisfies the following skein relation | ||
(see the corresponding `KnotInfo description page | ||
<https://knotinfo.math.indiana.edu/descriptions/jones_homfly_kauffman_description/polynomial_defn.html)>`__): | ||
<https://knotinfo.org/descriptions/jones_homfly_kauffman_description/polynomial_defn.html)>`__): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. remove There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed! |
||
|
||
.. MATH:: | ||
|
||
|
@@ -1547,7 +1547,7 @@ def jones_polynomial(self, variab=None, skein_normalization=False, puiseux=False | |
|
||
The Jones polynomial `V(L)` of a link `L` satisfies the following skein | ||
relation (see the corresponding `KnotInfo description page | ||
<https://knotinfo.math.indiana.edu/descriptions/jones_homfly_kauffman_description/polynomial_defn.html)>`__): | ||
<https://knotinfo.org/descriptions/jones_homfly_kauffman_description/polynomial_defn.html)>`__): | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. remove There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed! |
||
|
||
.. MATH:: | ||
|
||
|
@@ -2005,7 +2005,7 @@ def khovanov_polynomial(self, var1='q', var2='t', torsion='T', ring=None, origin | |
- :wikipedia:`Khovanov_homology` | ||
- :wikipedia:`Reduced_homology` | ||
- [ORS2013]_ | ||
- `KnotInfo <https://knotinfo.math.indiana.edu/descriptions/khovanov_unreduced_integral_polynomial.html>`__ | ||
- `KnotInfo <https://knotinfo.org/descriptions/khovanov_unreduced_integral_polynomial.html>`__ | ||
""" | ||
if not ring: | ||
if self.is_knot(): | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copy/paste error (3 times at least)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for reviewing! I've fixed it in the current commit.