Player statistics overhaul

[hburn7]

Feature Name

Overview

Update various DTOs related to player dashboard stats.

Dependencies

None.

Notes

To accurately test this, a local copy of statistics will be needed. To obtain this, you must do the following:

• Clone the otr-processor repo and set it up
• Setup the DataWorkerService appsettings configuration
• In the DataWorkerService config, disable OsuTrack and Osu under "Players" and enable "Tournaments"
• Run the script under scripts/sql/dev/accept-all-pre.sql
• Let the DataWorkerService run until all tournaments have a ProcessingStatus of 4.
• Stop the data worker, then run the otr-processor, it should generate ratings and stats for each verified tournament.
• Run the DataWorkerService again until all tournaments have a ProcessingStatus of 5.

Tasks

• Rename PlayerStatsDTO to PlayerDashboardStatsDTO
• In PlayerFrequencyDTO, remove all fields except Frequency and replace the removed fields with a PlayerCompactDTO representing the teammate or opponent.
• Change the PlayerDashboardStatsDTO to store an IEnumerable<PlayerRatingChartDataPointDTO> instead of a PlayerRatingChartDTO
• If it does not exist, create a MatchCompactDTO which contains basic information about a match (no navigations).
• Replace the int? MatchId field on RatingAdjustmentDTO with a new MatchCompactDTO? Match field.
• Add a Mods field to ModStatsDTO to indicate which mod it belongs to.
• Return a list of ModStatsDTO dynamically populated instead of a static DTO.
• Rename ModStatsDTO to PlayerModStatsDTO and document to reflect that this DTO should only be used to represent a player's mod stats rather than overall stats for the mod.

[myssto]

Rename PlayerStatsDTO to PlayerDashboardStatsDTO

why? If anything it should stay the same or change to just PlayerDTO

Rename PlayerFrequencyDTO to TeammateOpponentFrequencyDTO

feels unnecessary

Rename PlayerRatingChartDataPointDTO to PlayerRatingAdjustmentDTO

Super convoluted to name it like this especially with the proposed inheritance. A rating adjustment is a player rating adjustment, and there are no other variations so why would we add the modifier? The rating chart is constructed from a list of rating adjustments. In my opinion, just keep it simple and add a list of rating adjustments to the DTO. Pretty sure we don't need any additional info here that is specific to the chart

[hburn7]

1. The main idea behind refining the naming here is so that we can use PlayerStatsDTO later. I imagine one day we have a site-wide player statistics page with various "overall" stats about players in general.
2. Fair.
3. The chart UI should definitely display the name of the match at a minimum, that's the main reason why they're separate right now from what I can remember.

[myssto]

The chart UI should definitely display the name of the match at a minimum, that's the main reason why they're separate right now from what I can remember.

Adding a MatchCompactDTO to the existing rating adjustment DTO makes the most sense? Mirror the entity

[hburn7]

I suppose so yeah.

[hburn7]

Changed as follows:

- [ ] Remove `PlayerRatingChartDataPointDTO`
- [ ] If it does not exist, create a `MatchCompactDTO` which contains basic information about a match (no navigations).
- [ ] Replace the `int? MatchId` field on `RatingAdjustmentDTO` with a new `MatchCompactDTO? Match` field.