-
Notifications
You must be signed in to change notification settings - Fork 113
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
Game Attendance Data Availability? #68
Comments
@msussman I'll poke around and see what data MLB provides. |
I'm unable to find anything in the XML files around attendance, though the venue name is widely available. |
I'm digging around in their new stats API statsapi.mlb.com/docs but the lack of documentation isn't getting me very far. I did find attendance in their live game endpoint, buried in an info property, here's a Dodgers @ Marlins game in 2016
It might be available through some smaller endpoints as this one is ENORMOUS, but would require some more research. As it is, I'm thinking this would be a good feature for mlbgame v3 (Stats API usage) since this info doesn't appear to be available in their XML files. |
It's available through
|
@trevor-viljoen If you have time, could you go ahead and add that in a PR. I don't really have much time but I could get to it if you can't. |
@panzarino I'll try to find some time to do it this week. I'll also take a look at rawboxscore vs boxscore and see how different they are from each other. The fix might be as simple as using rawboxscore instead of boxscore. |
Is anyone working on this? I've done some work with MLB's |
Have at it, I could use another release soon. |
Which class should the |
@Pertempto I think that it would fit well with the other stats provided by the |
Great! I'm working on it now, and I'll probably have a pull request in the next few hours. |
I've implemented the attendance feature, but I was wondering if I should add all the top level attributes from |
@Pertempto It would be great if you could add that. It is fine to have duplicate attributes, just report everything. |
@panzarino Where am I supposed to "report everything"? Is it as simple as adding the new attributes to the |
I've created a pull request with all the new attributes. #71 |
Merged |
Thanks all. When are you planning on releasing a new version?
…On Sat, Mar 24, 2018, 3:47 PM Zach Panzarino ***@***.***> wrote:
Merged
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#68 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIEW6xH-GIbpdqnTpT0Qrg5JYbTn_eMQks5thqLfgaJpZM4Sa9QM>
.
|
I could definitely use a 2.5. Only missing piece from the LED board before the regular season starts is the probable starters which is just waiting for a new tag. |
I'll work on updating a few things and hopefully pushing out a new release in the coming days. |
Sounds great! Looking forward to checking out the update.
…On Sun, Mar 25, 2018 at 2:05 PM Zach Panzarino ***@***.***> wrote:
I'll work on updating a few things and hopefully pushing out a new release
in the coming days.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#68 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIEW63td8NwGtqNfUTM77JNJIkNMn4Jwks5th9xmgaJpZM4Sa9QM>
.
|
@msussman Sorry to disappoint, but I just looked at my schedule and I have almost no time this week so I'll have to push it back to sometime next week. |
No worries. I won't need it in earnest until next month
…On Mon, Mar 26, 2018, 3:46 PM Zach Panzarino ***@***.***> wrote:
@msussman <https://github.com/msussman> Sorry to disappoint, but I just
looked at my schedule and I have almost no time this week so I'll have to
push it back to sometime next week.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#68 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIEW67L835ZdkRcU5B8m2Vq5pc8PqsGWks5tiUWhgaJpZM4Sa9QM>
.
|
@panzarino, I just updated to the new release and am finding some issues with the attendance implementation.
|
@Pertempto could you look into this? |
@panzarino just updated my local repo, hopefully I'll be able to look at it in next week. |
Sorry, but I was busy last week and wasn't able to work on this. Someone else might want to do this because I can't promise that I'll have the time. |
Wanted to reach out to see if attendance data was available through the API
The text was updated successfully, but these errors were encountered: