Skip to content

[codex] Add graph cache directory option - #153

Merged
DanielSeemaier merged 2 commits into
mainfrom
codex/graph-cache-dir
Jun 8, 2026
Merged

[codex] Add graph cache directory option#153
DanielSeemaier merged 2 commits into
mainfrom
codex/graph-cache-dir

Conversation

@DanielSeemaier

Copy link
Copy Markdown
Member

Summary

Adds a --graph-cache-dir option to the shared-memory KaMinPar binary. When provided, KaMinPar checks whether the cache directory contains a graph file with the same basename as the -G/--graph input and uses the cached copy only if its file size matches the original input graph.

This lets benchmark invocations continue to reference graphs on a shared filesystem while opportunistically reading matching local SSD copies when they are available.

Validation

  • clang-format -i apps/KaMinPar.cc
  • git diff --check
  • cmake -S . -B build-codex -DCMAKE_BUILD_TYPE=Debug -DINSTALL_KAMINPAR=OFF -DKAMINPAR_BUILD_WITH_CCACHE=OFF
  • cmake --build build-codex --target KaMinParApp -j 8
  • CLI help smoke check for --graph-cache-dir and absence of --local-scratch
  • Cache-hit smoke test with a corrupted original and same-sized valid cached graph
  • Cache-miss smoke test with a wrong-sized cached graph falling back to the original

@DanielSeemaier
DanielSeemaier marked this pull request as ready for review June 8, 2026 13:49
@DanielSeemaier
DanielSeemaier merged commit 75e1ee0 into main Jun 8, 2026
27 checks passed
@DanielSeemaier
DanielSeemaier deleted the codex/graph-cache-dir branch June 8, 2026 14:05
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