seemethere / nba_py

Python client for NBA statistics located at stats.nba.com
BSD 3-Clause "New" or "Revised" License
1.05k stars 255 forks source link

Update player.py and team.py #110

Open inoble opened 6 years ago

inoble commented 6 years ago

I have found 24 endpoints from stats.nba.com that are not currently included in nba_py

Here are the first 6 new endpoints in 6 new classes that can hopefully be added to the master repo

I'm quite new to Python and GitHub so please let me know if I'm doing anything wrong, but hopefully I did enough testing on these classes before committing.

inoble commented 6 years ago

the 'Travis CI' check timed out after 10 minutes.

Travis CI's documentation does not seem to offer any reason why this has occurred, but I notice that it's happened to previous commits.

bttmly commented 6 years ago

The IP addresses Travis uses are blocked by nba.com https://github.com/seemethere/nba_py/issues/88

rneu31 commented 6 years ago

What are the future plans for this project if the tests don't pass? I'm willing and able to take on some of the monitoring of some of this stuff. @seemethere

inoble commented 6 years ago

Thanks @rneu31, it does seem a little quiet around here! I could help out if you need.

Is nba_py used by reddit.com/r/nba or any other websites? Do we have a list of websites that make use of the repo? I'm interested to find out more, it would be good to know contributions are going to be put to use.

BenikaH commented 6 years ago

I'm also willing to contribute.

inoble commented 6 years ago

If anybody else wants to help me out with these new 'endpoints', here are the endpoints I found that I'm yet to write up:

winprobabilitypbp teamandplayersvsplayers leagueplayerondetails leaguedashoppptshot infographicfanduelplayer franchiseplayers franchiseleaders fantasywidget defensehub boxscorematchups boxscoredefensive assisttracker assistleaders

mneedham91 commented 6 years ago

Hi @inoble , I am taking a stab at adding the endpoints you found. I'm curious, how did you find these? Also a question for you and anyone else watching this thread, any advice for decoding some of these variables? For example, endpoint winprobabilitypbp requires variable runtype which I have no idea what it does or what options are available.

BenikaH commented 6 years ago

Hi @inoble, I may have some free time this upcoming week. I’m open to helping out where I can.

BenikaH commented 6 years ago

I’ve added features for leaguedashoppptshot already. I can merge my branch soon.

inoble commented 6 years ago

@mneedham91 I stalked @seemethere's reddit comments and saw that he found the original 'endpoints' by checking a javascript file on nba.com. So I did the same and extracted all the endpoints.

The problem is I can't find it anymore! I just had a pretty good look again and had no luck.

BenikaH commented 6 years ago

@inoble How's it coming with those endpoints? Anything you need help with?

inoble commented 6 years ago

@BenikaH - I've not been working on this any further since February.

The endpoints that still need coding are:

winprobabilitypbp teamandplayersvsplayers leagueplayerondetails leaguedashoppptshot infographicfanduelplayer franchiseplayers franchiseleaders fantasywidget defensehub boxscorematchups boxscoredefensive assisttracker assistleaders

I might come back to this in the coming months because I might be trying to code my own simulation engine now that my sim league (dynastyfive.com) is ready to start its off season, but I can't guarantee it.