[BUG] Broken links in docs

[cakiki]

Clicking on any vectorizer here in the functions column leads to a 404 page: https://vectorizers.readthedocs.io/en/latest/sequence_taxonomy.html#Vectorizer-Functions

Some of the hyperlinks also include a weird space in them (e.g., KDEV ectorizer) even though that's not in the original notebook.

I'd submit a PR but I have no idea how to fix this.

[lmcinnes]

I think it's because the API docs got broken at some point -- which in turn is due to sphinx autodoc which I don't really understand that well. I'll see if I can figure out something.

[cakiki]

@lmcinnes Abusing this issue for a related question so as not to open a new one: how should I go about sending PRs to a notebook? Should I re-run the whole thing locally, or rather just edit it as a JSON?

[lmcinnes]

Best to just rerun the whole thing logically. That way if the library has changed and something is broken it can get fixed in the notebook.

…

On Fri, Nov 15, 2024 at 7:50 AM Christopher Akiki ***@***.***> wrote: @lmcinnes <https://github.com/lmcinnes> Abusing this issue for a related question so as not to open a new one: how should I go about sending PRs to a notebook? Should I re-run the whole thing locally, or rather just edit it as a JSON? — Reply to this email directly, view it on GitHub <#139 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AC3IUBNXT2LG25J3TBROUH32AXUXZAVCNFSM6AAAAABRK6TYZ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZYG42DEOBQHA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[kalebruscitti]

I just ran into this problem too. I learned how to do Sphinx when I wrote the IWT doc and the dbmapper doc, so I can probably figure out how to fix these links. I'll do it next time I need a break from my actual projects.