-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
ff_scoringhistory() and return fumbles for Sleeper #281
Comments
Hm! That’s interesting. Will try to see if nflfastr has return fumbles |
Hmm, it seems like they don’t have it, unfortunately: https://www.nflfastr.com/reference/calculate_player_stats.html I’ll see if I can raise an issue and/or help fix it over there first and then can revisit this issue |
Good news - they do! I was actually able to pull the fumble I referenced specifically. Not sure how easy this is to incorporate into existing functionality but check this out: lar_phi <- nflfastR::build_nflfastR_pbp(game_ids = "2020_02_LA_PHI") lar_phi <- filter(lar_phi, fumbled_1_player_id == "00-0033908") lar_phi_t <- transpose(lar_phi) Also I admit this is a bit of a hunch as I wasn't able to easily verify this for other games. |
Oh! I see it's an issue with their stat aggregation. I assumed you were doing some wizardry with the raw pbp data. I suppose I could work around this by creating a separate dataset of just kick/fumble returns based on build_nflfastR_pbp()? |
Yup, that’s definitely a way to get around it for now, I suspect that the nflfastr team would be happy to take in requests for these sort of aggregations |
Tacked on a request at nflverse/nflfastR#235, when that gets resolved I can re-map here. |
Brief update as of 2022-12-11: still pending on fumble recovery/fumble returns being included in |
Describe the bug
Discovered when comparing aggregate ff_scoringhistory() results by team that ff_scoringhistory() appears to be ignoring return fumbles. This is a pretty obscure scoring thing that I think may be unique to Sleeper. See: https://support.sleeper.app/en/articles/4056849-why-did-my-player-lose-points-for-a-special-teams-fumble
Reprex
Please provide a short, reproducible example of the code you are trying to run.
For more on reprexes, check out https://reprex.tidyverse.org
# Please include the code used for the connection object from `ff_connect()`
Expected behavior
Return fumbles to hit ff_scoringhistory() same as any other fumble
Session information
Please copy the output of
sessionInfo()
into the box below.Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
For reference, check out nflfastR data for Eagles/Rams, 2020. Cooper Kupp fumbled on a punt return, and I can verify that ff_scoringhistory() is 2.0 pts higher for my league than his official score in the Sleeper app.
The text was updated successfully, but these errors were encountered: