Skip to content

Fix: catch 404s but ignore time out errors #483

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lwasser
Copy link
Member

@lwasser lwasser commented May 8, 2025

I think we should be catching 404s but ignoring timeout errors in CI. let's see if that helps

@lwasser
Copy link
Member Author

lwasser commented May 8, 2025

Okay, this caught some legitimate 404 errors 🚀 and the timeout errors (0) are gone. Let's not merge this until I fix the broken links. Then, maybe we can remove footnotes altogether to avoid the translation issues. I'm not sure footnotes are worth it in tutorial content.

@lwasser lwasser marked this pull request as draft May 8, 2025 23:57
@sneakers-the-rat
Copy link
Contributor

Heck ya.

I think it would be worth diagnosing the footnotes problem - it indicates that something is wrong with how markdown is being parsed in the translated sites, and we wouldn't want that to e.g. break some other kind of markdown element without us knowing about it.

I need to take a look at how/when myst parser is evoked relative to the gettext injection, bc it's super odd that the target footnote rendered and even had the correct backlink, but the forward link wasn't rendered.

@lwasser
Copy link
Member Author

lwasser commented May 11, 2025

@sneakers-the-rat, thank you! If you have any ideas as to what is happening, that would be great (regarding the footnotes!). I keep looking at it, and I'm not sure how to debug this issue as it's specific to the translations.

And yes! It's really odd that the target is ok, and it's struggling with the footnotes. I hadn't considered that this could be a myst issue! I could post a bug report tomorrow in the myst repo to see if they have any ideas.

@sneakers-the-rat
Copy link
Contributor

Searching issues in myst-parser, found one that looks like the same problems: executablebooks/MyST-Parser#690

and someone had already backlinked to a previous issue of ours where @flpm had resolved it with a venv version update: #390

So maybe there were two bugs in #390? This footnote rendering bug (that has a patch in that issue) as well as the venv version problem?

@flpm
Copy link
Member

flpm commented May 12, 2025

It makes sense, I wonder though how the issue I was having magically fixed itself.

Not sure if related but I just noticed that our build book CI action is running on a Python 3.9, maybe that forces older versions of the sphinx modules?

@flpm
Copy link
Member

flpm commented May 12, 2025

Maybe the bug happens for a certain combination of Sphinx module version. That happened in my outdated venv and it also happens in CI because of Python 3.9? Just a crazy theory, I am away from my computer now so I can check 😞

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.

3 participants