MattDMo / NYTimesArticleAPI

Python wrapper for the New York Times Article Search API
MIT License
11 stars 13 forks source link

pip installation broken #2

Open GuessWhoSamFoo opened 7 years ago

GuessWhoSamFoo commented 7 years ago

I'm not sure if this API wrapper is still being actively maintained.

Running pip install NYTimesArticleAPI results in this error.

Collecting NYTimesArticleAPI
  Could not find a version that satisfies the requirement NYTimesArticleAPI (from versions: )
No matching distribution found for NYTimesArticleAPI

EDIT: Temporary workaround by installing via pip locally

pip install -e <Path to NYTimesArticleAPI>
prateeky2806 commented 7 years ago

The pip install is not working and while installing pip locally I am getting an error

"python setup.py egg_info" failed with error code 1 in /tmp/pip-ADB9WL-build/

Can anyone please fix this or suggest me some way to install this package.

GuessWhoSamFoo commented 7 years ago

Did you try updating setuptools yet?

prateeky2806 commented 7 years ago

@GuessWhoSamFoo
Yes, I have tried all the thing mentioned on this page including upgrading setuptools. Can it be some problem with C compiler? I am using bash on windows but also tried it on ubuntu and it didn't work for me.

GuessWhoSamFoo commented 7 years ago

@prateeky2806 Hmm... I doubt it is the C compiler. Maybe try running sudo pip , that works sometimes. Other than that, I would check the logs to see if any dependencies are missing.

nkramer44 commented 7 years ago

Did anyone ever figure out why pip install isn't working? The package shows up in pip search, but can't be installed for the same reason as @prateeky2806 was having. setuptools and requests are both updated

GuessWhoSamFoo commented 7 years ago

Not really sure. I just did an install on a new virtual environment using Anaconda on Ubuntu 16.04 without any issues.

ChampeBarton commented 6 years ago

Has anybody figured this one out yet? Setuptools and requests are both updated and I'm getting

Could not find a version that satisfies the requirement nytimesarticleapi (from versions: ) No matching distribution found for nytimesarticleapi

The package shows up in a pip install search, and shows up online @ https://pypi.python.org/pypi/nytimesarticleapi, too; however, that link says the package was authored by Brendan Ryan and his github homepage gives a 404error. Could really use some help!

I also notice in Matt's Progress Tracking he has yet to check off "upload to PyPI" ... could that be the problem?

people-can-fly commented 6 years ago

Basically i copied the code from NYTimesArticleAPI/NYTimesArticleAPI/search_api.py and replaced it with my installed nytimesarticle file nytimesarticle.py

Thus it removed

def _utf8_encode(self, d): ......

which prevented nytimesarticle module to work with python3, throwing TypeError:must be str,not bytes on search function of the api.

cbjrobertson commented 6 years ago

Has anyone solved this?

Update: I replaced the nytimesarticle file nytimesarticle.py with NYTimesArticleAPI/NYTimesArticleAPI/search_api.py and it seems to be working fine, with no Type Error: must be str, not bytes. I don't know if this is a good solution, but it's working for me.

ChampeBarton commented 6 years ago

Not positive if this problem has been solved, but I ended up forking the whole package then essentially remaking the package with different identifying materials and loading it as a separate package onto my local drive. Check my github under my thesis source code to see if that provides any help.

Best, Champe

On Feb 21, 2018, at 4:25 PM, cbjrobertson notifications@github.com wrote:

Has anyone solved this?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/MattDMo/NYTimesArticleAPI/issues/2#issuecomment-367477461, or mute the thread https://github.com/notifications/unsubscribe-auth/AKfIYBZCRztHyZEAYDx_ElTf4RecBtxYks5tXIm7gaJpZM4Mkx1C.

deveArt commented 5 years ago

This ModuleNotFoundError: No module named 'ez_setup'

caused an installation error: pip._internal.exceptions.InstallationError: Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-s52oqssv/NYTimesArticleAPInew/

installation of ez_setup doesn't help. I think it should be removed from setup.py

lud0 commented 5 years ago

@deveArt the same for me, I've removed it in my fork and now it's working.