wehoop icon indicating copy to clipboard operation
wehoop copied to clipboard

text_type in load_wbb_pbp has MadeFreeThrow for both made and missed free throws

Open sambruchhaus opened this issue 3 years ago • 1 comments

Describe the bug The column text_type in the resulting data from load_wbb_pbp() has "MadeFreeThrow" for both made and missed free throws

To Reproduce Steps to reproduce the behavior:

  1. Call load_wbb_pbp()
  2. Filter by text to retrieve missed free throws
  3. text for "[Name] missed Free Throw" has "MadeFreeThrow" for the value of text_type

Expected behavior The text should be either "FreeThrow" or "MissedFreeThrow" for more intuitive filtering.

Screenshots If applicable, add screenshots to help explain your problem.

Screen Shot 2022-01-11 at 6 34 16 PM

Version: 1.2.1

Additional context Encountered while trying to calculate Athletes Unlimited Scoring from Play-by-Play

sambruchhaus avatar Jan 11 '22 23:01 sambruchhaus

While not something I mess with to create the type_text field, nonetheless disconcerting. I will try to validate more as the season goes along (more like off-season) , but if you have like code to validate the types, I'd welcome a PR

saiemgilani avatar Jan 12 '22 00:01 saiemgilani