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

Some strangeness with possessions/game #142

Open
1 of 2 tasks
Alex-At-Home opened this issue Dec 22, 2020 · 9 comments
Open
1 of 2 tasks

Some strangeness with possessions/game #142

Alex-At-Home opened this issue Dec 22, 2020 · 9 comments
Labels
bug Something isn't working

Comments

@Alex-At-Home
Copy link
Owner

Alex-At-Home commented Dec 22, 2020 •
edited
Loading

  • First off, calling it ppg in tooltips is v confusing! (renamed to poss/g in 2522ab1)
  • It seems wrong - see below FIXED but need to reimport all the data and then remove the "don't calc tempo" back

For 2020/21 W Baylor, they've played 8 games, so there should be 320 minutes

Instead there are 400 ... and 422 of them are the sum of positive duration_mins vs -22 as the sum of 13 negative duration_mins stints
@Alex-At-Home

This comment has been minimized.

Sign in to view
@Alex-At-Home Alex-At-Home added the bug Something isn't working label Dec 22, 2020
@Alex-At-Home

This comment has been minimized.

Sign in to view
@Alex-At-Home

This comment has been minimized.

Sign in to view
Alex-At-Home pushed a commit that referenced this issue Dec 22, 2020
Pull poss/g from site due to issues described in #142
2522ab1
@Alex-At-Home

This comment has been minimized.

Sign in to view
Alex-At-Home pushed a commit that referenced this issue Jan 8, 2021
[#142] Put tempo back except for specified teams
52ee740
@Alex-At-Home

This comment has been minimized.

Sign in to view
@Alex-At-Home

This comment has been minimized.

Sign in to view
Alex-At-Home pushed a commit to Alex-At-Home/cbb-explorer that referenced this issue Jan 8, 2021
Add robustness to our of order events causing duration errors, see Al…
17e813b 
…ex-At-Home/cbb-on-off-analyzer#142
Alex-At-Home pushed a commit to Alex-At-Home/cbb-explorer that referenced this issue Jan 9, 2021
Fix women stint start time calcs, see Alex-At-Home/cbb-on-off-analyze…
9925eeb 
…r#142
@Alex-At-Home Alex-At-Home mentioned this issue Jan 9, 2021
Open
6 tasks
@Alex-At-Home

This comment has been minimized.

Sign in to view
Alex-At-Home pushed a commit to Alex-At-Home/cbb-explorer that referenced this issue Jan 10, 2021
Add more robustness to all stint start time calcs, see Alex-At-Home/c…
1ddf8e9 
…bb-on-off-analyzer#142
@Alex-At-Home

This comment has been minimized.

Sign in to view
@Alex-At-Home
Copy link
Owner Author

Actually I remember now .. added ask to top of OP

@Alex-At-Home Alex-At-Home reopened this Apr 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Alex-At-Home