8000 Trivial: Fix broken link for Peephole Optimizer Paper by banool · Pull Request #235 · python/devguide · GitHub
[go: up one dir, main page]

Skip to content

Trivial: Fix broken link for Peephole Optimizer Paper #235

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

Merged
merged 2 commits into from
Jun 29, 2017

Conversation

banool
Copy link
Contributor
@banool banool commented Jun 25, 2017

The old link for this paper is broken (or at least doesn't link to the paper in question). A quick google search revealed a few other places where the paper can be found. I opted for the link on legacy.python.org but any of the other links could be used if they better suit the documentation style.

I've run make html and it seems to have worked fine 😄

@serhiy-storchaka
Copy link
Member

I'm not sure that legacy.python.org will not disappear one time. Maybe @smontanaro have more stable link to his paper?

@smontanaro
Copy link
Contributor

I took down www.smontanaro.net. If you like (and want to avoid adding legacy.python.org references), I can put a copy up on Google Drive or Dropbox, or just send you a copy you can drop into place on the website itself.

@brettcannon
Copy link
Member

A link to Drive or Dropbox would also work.

@serhiy-storchaka
Copy link
Member

We could also just move the paper to non-legacy python.org.

@smontanaro
Copy link
Contributor
smontanaro commented Jun 26, 2017 via email

@smontanaro
Copy link
Contributor
smontanaro commented Jun 26, 2017 via email

@brettcannon
Copy link
Member

@smontanaro I can view it (it's raw HTML instead of rendered HTML, but since this is a historical document I think that's fine).

@smontanaro
Copy link
Contributor
smontanaro commented Jun 26, 2017 via email

@brettcannon brettcannon merged commit 5ae2b57 into python:master Jun 29, 2017
@brettcannon
Copy link
Member

Thanks everyone for helping out!

AA-Turner pushed a commit to AA-Turner/devguide that referenced this pull request Jun 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0