Skip to content

Use list instead of dict for breakdown#304

Open
congwang09 wants to merge 4 commits intomainfrom
use-list-breakdown
Open

Use list instead of dict for breakdown#304
congwang09 wants to merge 4 commits intomainfrom
use-list-breakdown

Conversation

@congwang09
Copy link
Copy Markdown
Contributor

@congwang09 congwang09 commented Apr 22, 2026

Relates to: atlanticwave-sdx/sdx-controller#524

Use list instead of dict for breakdown, a dict collapses paths such as tenet -> sax -> tenet into one tenet segment, which corrupts alternate-path breakdown generation.

@coveralls
Copy link
Copy Markdown

coveralls commented Apr 22, 2026

Coverage Report for CI Build 24919871866

Coverage increased (+0.1%) to 90.43%

Details

  • Coverage increased (+0.1%) from the base build.
  • Patch coverage: 22 of 22 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1598
Covered Lines: 1466
Line Coverage: 91.74%
Relevant Branches: 774
Covered Branches: 679
Branch Coverage: 87.73%
Branches in Coverage %: Yes
Coverage Strength: 1.83 hits per line

💛 - Coveralls

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.

2 participants