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.

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

Scrape Individual Savant Player Pages #354

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TravisGibbs
Copy link

I recently started using this library which has greatly aided my work. I noticed a common pattern where I wanted to gather a players year over year stats, such as their position on exit velo leaderboards over years. I couldn't find a way to do this without making multiple queries and then filtering and merging dfs. Luckily this data is available on each player's savant page so I built functionality to gather the tables on those pages for pitcher and batter data.

I added testing and followed the instructions on the contributing.md. My only concern is linking the columns provided by the pandas reading of the html. The initial implementation below is done by matching column names which seems reasonable to me, but it would also be possible to manually parse these tables.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant