Skip to content

feat(players): overlay skill history on ELO chart#592

Merged
garrappachc merged 3 commits into
masterfrom
feat/elo-chart-skill-overlay
May 28, 2026
Merged

feat(players): overlay skill history on ELO chart#592
garrappachc merged 3 commits into
masterfrom
feat/elo-chart-skill-overlay

Conversation

@garrappachc
Copy link
Copy Markdown
Member

Summary

  • Adds a skill layer on top of the ELO history chart so admins can correlate ELO changes with manual skill adjustments
  • Skill is rendered as a dashed white stepped line on a right-side Y-axis, aligned to existing ELO data points
  • The overlay only appears when the selected class has admin-set skill history; charts for players without skill data are unchanged

How it works

For each ELO data point, buildSkillData finds the most recent skillHistory entry whose at date is ≤ the game date, producing a parallel array of number | null. This gives a step-function that stays flat between admin changes and jumps when skill is updated.

Test plan

  • Open a player profile with skill history set — verify the dashed skill line appears on the ELO chart with a right-side axis
  • Switch between class tabs — verify skill line updates correctly per class
  • Switch between "Over time" / "By game #" modes — verify both axes render correctly
  • Open a player profile with no skill history — verify chart looks identical to before (no second axis, no legend)

🤖 Generated with Claude Code

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Playwright test results

passed  199 passed

Details

stats  199 tests across 29 suites
duration  8 minutes, 33 seconds
commit  ef40cf4

garrappachc and others added 2 commits May 27, 2026 23:00
Adds a second dataset to the ELO history chart that shows the player's
admin-set skill value over time, aligned with ELO data points. The skill
line uses a dashed stepped style on a right-side Y-axis so ELO and skill
changes can be correlated visually. The overlay is only rendered when
skill history exists for the selected class.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@garrappachc garrappachc force-pushed the feat/elo-chart-skill-overlay branch from e9ae9b9 to 046ccba Compare May 27, 2026 21:00
@garrappachc garrappachc merged commit de3d44b into master May 28, 2026
25 checks passed
@garrappachc garrappachc deleted the feat/elo-chart-skill-overlay branch May 28, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant