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

Updated Weekly Luck Record function to also calculate and report Season Luck Record and Ranking/Place #227

Closed

Conversation

cdpeca
Copy link
Contributor

@cdpeca cdpeca commented Oct 15, 2024

This feature enhances the Team Luck Rankings section by adding a Season Record for Luck in addition to the Weekly Record for Luck. On top of that it also calculates the Season Ranking/Place for the Season Record. With this feature the last column of the Team Luck Rankings now provides 3 distinct data points:

  • Weekly Luck Record
  • Season Luck Record
  • Season Luck Rank

This allows for a very useful analysis of what a record would look like for everyone in a league if you eliminated the positive/negative luck. Probably the most useful comparison that can be made of what a team's record would really look like compared to everyone else if you eliminate luck as a variable.

@cdpeca cdpeca force-pushed the feature(luck_season_record_with_rank) branch 2 times, most recently from b193ff0 to f5d3ff3 Compare October 22, 2024 22:59
@cdpeca cdpeca force-pushed the feature(luck_season_record_with_rank) branch from f5d3ff3 to 6d34885 Compare October 22, 2024 23:03
@cdpeca
Copy link
Contributor Author

cdpeca commented Oct 22, 2024

Rebased with upstream/main commits and resolved conflict on report table header. Should now be good to merge this feature branch into upstream/main

@cdpeca cdpeca closed this Nov 13, 2024
@cdpeca cdpeca deleted the feature(luck_season_record_with_rank) branch November 13, 2024 06:59
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