jldbc / pybaseball

Pull current and historical baseball statistics using Python (Statcast, Baseball Reference, FanGraphs)
MIT License
1.23k stars 330 forks source link

Install fails #212

Closed pfe1223 closed 3 years ago

pfe1223 commented 3 years ago

Trying to install the package, but am getting the error below.

Screen Shot 2021-05-05 at 9 27 55 PM

tjburch commented 3 years ago

This could be any number of problems, generally external to pybaseball - can you pip install other packages?

What command are you using to install? It's not shown in your screenshot.

Try running python -c "from Cython.Distutils import build_ext". If that throws an error, it means there's something wrong with your cython install, probably just run a pip install --upgrade cython. If it doesn't throw an error, my guess would be that sudo isn't picking up the right cython, see this stackoverflow answer

pfe1223 commented 3 years ago

I cloned the repo, changed to the appropriate directory, and typed pip3 install -e .. I installed cython but got an error about pyarrow. Somebody on StackOverflow suggested installing cmake, which I did. I am still getting an error about pyarrow.

Screen Shot 2021-05-06 at 1 41 44 PM

pfe1223 commented 3 years ago

I got it working, though more through stumbling around with pip install than anything. Thanks for your help.

tjburch commented 3 years ago

Was it something system-based? Would be nice to know if there was something off with our requirements (though I thought the ci would have caught that)

pfe1223 commented 3 years ago

Things that I did (not sure which of these solved the issue):

tjburch commented 3 years ago

Cool, thanks. I think this should be fine looking at setup.py. @schorrm, this can probably be closed.