Skip to content

Merge compact degree maps; trim CI fanout - #76

Merged
adsharma merged 2 commits into
mainfrom
fix/ci-trigger-fanout
Sep 11, 2026
Merged

adsharma merged 2 commits into
mainfrom
fix/ci-trigger-fanout

Conversation

@adsharma

Copy link
Copy Markdown
Contributor

Two commits:

  1. fix(graph): merge compact in/out degree maps into one flat_map (PR 74 review: same keys, saves n*sizeof(node), halves sorted-key maintenance). Also installs *.tpp headers. C++ tests: 65 InducedSubgraph/FlatMap tests pass.
  2. ci: stop redundant workflow fanout — ci-matrix build job skips push-to-main (already PR-validated), plus cancel-in-progress for superseded PR/smoke runs. PRs run 1 workflow, merges 2, releases wheels only.

InDegree and outDegree share the same keys, so storing them together
saves n * sizeof(node) bytes and halves the sorted-key maintenance.
Also install *.tpp headers so FlatMap.tpp ships with the headers.
- ci-matrix 'build' job no longer runs on push to main (already
  validated as a PR); post-merge coverage comes from release-build
  plus the wheel smoke build.
- Cancel superseded runs: PR updates cancel older PR runs, and a
  newer main cancels an older wheel smoke build. Release runs never
  cancel each other.
Result: PRs run 1 workflow, merges run 2 (was 3 with a duplicated
check build), releases run wheels only.
@adsharma
adsharma merged commit 1065e16 into main Sep 11, 2026
6 checks passed
@adsharma
adsharma deleted the fix/ci-trigger-fanout branch September 11, 2026 21:02
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.

1 participant