-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Allow name_only option gensim downloader api #2143
Allow name_only option gensim downloader api #2143
Conversation
This reverts commit 6c06fbc.
gensim/downloader.py
Outdated
@@ -29,6 +29,7 @@ | |||
Also, this API available via CLI:: | |||
|
|||
python -m gensim.downloader --info <dataname> # same as api.info(dataname) | |||
python -m gensim.downloader --info_name_only # same as api.info(name_only=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better to do it as parameter of --info
flag I think (instead of new --info_*
flag), like --info name
changes made @menshikh-iv |
gensim/downloader.py
Outdated
@@ -29,6 +29,7 @@ | |||
Also, this API available via CLI:: | |||
|
|||
python -m gensim.downloader --info <dataname> # same as api.info(dataname) | |||
python -m gensim.downloader --info name_only # same as api.info(name_only=True) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--info name
please :) (but stay name_only
parameter for CLI)
@aneesh-joshi thanks! |
Currently, to get the exact names of the models or corpora, a user has to either:
gensim.info()
and look through the huge json dump to get the exact namesWhen using gensim-data, I often forget the exact key.
"Was it
'glove-wiki-gigaword'
or'glove-gigaword-wiki'
?"It would be very helpful if a user could, in the terminal or otherwise, type:
gensim.info(name_only=True)
orpython -m gensim.downloader --info_name_only
ans get the following output:
Notes:
The current develop's downloader.py is failing the doctests without me doing anything.