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.

Sign up for GitHub

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

Jump to bottom

league leaders function giving some errors #12

Closed
pwu97 opened this issue Aug 28, 2019 · 2 comments
Closed

league leaders function giving some errors #12

pwu97 opened this issue Aug 28, 2019 · 2 comments
Assignees
@toddrob99
Labels
mlb data inconsistency

Comments

@pwu97
Copy link

pwu97 commented Aug 28, 2019

i was going through some of the code in the documentation:

this line of code:
print( statsapi.league_leaders('onBasePlusSlugging',statGroup='hitting',limit=5) )

gives:
IndexError: list index out of range
@toddrob99
Copy link
Owner

I just noticed this last night. Apparently StatsAPI stopped supporting the statType of statsSingleSeason. I have made changes to pull career leaders in the examples instead, since I no longer see a way to pull single season record holders. I need to do some more testing before publishing.

@toddrob99 toddrob99 self-assigned this Aug 28, 2019
toddrob99 added a commit that referenced this issue Aug 29, 2019
@toddrob99
Add functions to return data instead of only formatted text, and upda…
6a1e45c 
…te league_leaders() to work around StatsAPI removal of support for statType=statsSingleSeason (#12)
@toddrob99 toddrob99 mentioned this issue Aug 30, 2019
Merged
@toddrob99
Copy link
Owner

toddrob99 commented Aug 30, 2019 •
edited

Please update to v0.1.0 via pip install --upgrade MLB-StatsAPI and thank you for reporting the issue

@skellet0r skellet0r mentioned this issue Oct 5, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@toddrob99 toddrob99

Labels
mlb data inconsistency
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@toddrob99 @pwu97