Skip to content
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

Broken file link in run_corpora_and_vector_spaces tutorial #2872

Closed
piskvorky opened this issue Jul 5, 2020 · 6 comments
Closed

Broken file link in run_corpora_and_vector_spaces tutorial #2872

piskvorky opened this issue Jul 5, 2020 · 6 comments
Labels
bug Issue described a bug difficulty easy Easy issue: required small fix documentation Current issue related to documentation
Milestone

Comments

@piskvorky
Copy link
Owner

Problem description

The run_corpora_and_vector_spaces.ipynb tutorial depends on a file on the web, and that file is missing.

Steps/code/corpus to reproduce

See https://groups.google.com/g/gensim/c/nX4lc8j0ZO0

Versions

Please provide the output of:

import platform; print(platform.platform())
import sys; print("Python", sys.version)
import numpy; print("NumPy", numpy.__version__)
import scipy; print("SciPy", scipy.__version__)
import gensim; print("gensim", gensim.__version__)
from gensim.models import word2vec;print("FAST_VERSION", word2vec.FAST_VERSION)

Unknown (probably any).

@piskvorky piskvorky added bug Issue described a bug documentation Current issue related to documentation difficulty easy Easy issue: required small fix labels Jul 5, 2020
@piskvorky
Copy link
Owner Author

piskvorky commented Jul 5, 2020

I put the file back for now.

Not sure whether we need that file on the web (as opposed to creating it directly within the notebook – the file is trivial). CC @mpenkov

The download link in the notebook seems broken too, at least when viewed on github:
Screenshot (21)

@piskvorky
Copy link
Owner Author

Fixed by #3148.

@raffaem
Copy link

raffaem commented Jun 6, 2021

The research paper here:
https://nlp.fi.muni.cz/~xrehurek/nips/rehurek_nips.pdf
is also on 404

@piskvorky
Copy link
Owner Author

piskvorky commented Jun 6, 2021

You're right :( Could you update that link to https://arxiv.org/abs/1102.5597 please?

10 years seems to the half-life of functional hyperlinks in academia.

@raffaem
Copy link

raffaem commented Jun 6, 2021

On this webpage:

https://radimrehurek.com/gensim/wiki.html#wiki

The Hoffman reference is broken both in pdf and in code

@piskvorky
Copy link
Owner Author

piskvorky commented Jun 6, 2021

Is that the one fixed in #3133?
EDIT: actually that PR updated the Hoffman reference in ldamulticore.py, but not in wiki.html.

@mpenkov let's plan the next Gensim release, the fixes and reports are starting to overlap :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue described a bug difficulty easy Easy issue: required small fix documentation Current issue related to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants