Skip to content
This repository was archived by the owner on Oct 11, 2026. It is now read-only.

fix(analytics): align EI defense comparison aliases - #109

Merged
Roddygithub merged 2 commits into
mainfrom
fix/ei-compare-defense-aliases
Aug 1, 2026
Merged

Roddygithub merged 2 commits into
mainfrom
fix/ei-compare-defense-aliases

Conversation

@Roddygithub

Copy link
Copy Markdown
Owner

Summary

  • keep the first anonymous agent for duplicate EI instance IDs
  • count defense deaths across all aliases for the same instance ID
  • prefer combat outcome down counts for named players when available
  • add regression coverage for duplicate anonymous instance IDs and outcome-vs-buff down counts

Checks

  • uv run pytest libs/gw2_analytics/tests/test_ei_compare.py libs/gw2_analytics/tests/test_buff_state.py -q
  • uv run ruff check libs/gw2_analytics/src/gw2_analytics/ei_compare.py libs/gw2_analytics/tests/test_ei_compare.py
  • uv run ruff format --check libs/gw2_analytics/src/gw2_analytics/ei_compare.py libs/gw2_analytics/tests/test_ei_compare.py
  • uv run python scripts/ei-parity/ei_diff.py --json /tmp/opencode/ei-20260506-before-pr.json --show "defenses|players.name" 20260506-225114
  • uv run python scripts/ei-parity/ei_diff.py --json /tmp/opencode/ei-20260429-before-pr.json --show "defenses|players.name" 20260429-232621
  • uv run python scripts/ei-parity/ei_diff.py --json /tmp/opencode/ei-20260125-before-pr.json --show "defenses|players.name" 20260125-001308

Signed-off-by: Roddy <roddy@users.noreply.github.com>
Signed-off-by: Roddy <roddy@users.noreply.github.com>
@Roddygithub
Roddygithub merged commit 65cebfd into main Aug 1, 2026
11 checks passed
@Roddygithub
Roddygithub deleted the fix/ei-compare-defense-aliases branch August 1, 2026 15:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant