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

fix(ei-compare): keep team on character-swap split slices - #205

Merged
Roddygithub merged 2 commits into
mainfrom
fix/parser-parity-after-190
Aug 9, 2026
Merged

Roddygithub merged 2 commits into
mainfrom
fix/parser-parity-after-190

Conversation

@Roddygithub

Copy link
Copy Markdown
Owner

Two independent EI-parity fixes reducing corpus deltas 190 -> 179.

Changes

1. EI compare — team on character-swap split slices

EI reports the original team on every slice when one account switches characters mid-fight (same instance_id/agent_id, new display name). Track known account names and apply the team when the account has more than one name or the slice is the last same-character slice.

  • libs/gw2_analytics/src/gw2_analytics/ei_compare.py + 2 tests

2. Rotation — expected duration for pending Resurrect casts

Resurrect (skill 1066) activations open with raw duration 700ms but the cast stays pending and EI reports the expected 800ms. Added to _PENDING_EXPECTED_DURATION_SKILLS.

  • libs/gw2_analytics/src/gw2_analytics/rotation.py + 1 test

Verification

  • uv run pytest libs/gw2_analytics/tests/test_rotation.py libs/gw2_analytics/tests/test_ei_compare.py all pass
  • uv run ruff check / uv run ruff format --check clean
  • Full EI-parity corpus: 189 -> 179 diffs (rotation 68 -> 58, zero Resurrect diffs)

Signed-off-by: Roland Salardon <r.salardon@gmail.com>
Signed-off-by: Roland Salardon <r.salardon@gmail.com>
@Roddygithub
Roddygithub force-pushed the fix/parser-parity-after-190 branch from 5ebf579 to c47e0eb Compare August 9, 2026 16:05
@Roddygithub
Roddygithub merged commit d378c2c into main Aug 9, 2026
11 checks passed
@Roddygithub
Roddygithub deleted the fix/parser-parity-after-190 branch August 9, 2026 16:11
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