Skip to content

Conversation

@Iglesys347
Copy link
Contributor

Reference Issues/PRs

#23631

What does this implement/fix? Explain your changes.

The link http://users.jyu.fi/~samiayr/pdf/ayramo_eurogen05.pdf is working properly, added it to the linkcheck_ignore list in scikit-learn/doc/conf.py

@Iglesys347 Iglesys347 changed the title Link works fine, added it to linkcheck_ignore DOC Link works fine, added it to linkcheck_ignore Jun 18, 2022
@lesteve
Copy link
Member

lesteve commented Jun 20, 2022

Thanks a lot for the PR!

Indeed the link works fine. I have other logs of make linkcheck where this links was fine as well. I am going to guess there was a temporary issue with the website when I ran make linkcheck.

For now, I am not going to merge this PR. In case this link is flagged as broken from time to time by make linkcheck, I will merge this PR together with a comment explaining that from time to time it leads to a timeout

@lesteve
Copy link
Member

lesteve commented Jun 20, 2022

I ran make linkcheck a few times and I can confirm that this links does not show up as broken, so I am going to close this one.

I suspect this was a temporary website issue when I ran the make linkcheck and created the issue, sorry for this!

Closing this one.

@lesteve lesteve closed this Jun 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants