Repository navigation
Add coverage audit profiles with GFQL lock-ins - #1518
Merged
Merged
Conversation
lmeyerov
force-pushed
the
codex/issue-1517-gfql-coverage-audit
branch
5 times, most recently
from
May 18, 2026 05:24
daef452 to
b061ac4
Compare
lmeyerov
force-pushed
the
codex/issue-1517-gfql-coverage-audit
branch
2 times, most recently
from
May 18, 2026 06:02
9f5b944 to
dba9bd0
Compare
lmeyerov
force-pushed
the
codex/issue-1517-gfql-coverage-audit
branch
from
May 18, 2026 20:24
dba9bd0 to
c00e2d1
Compare
lmeyerov
force-pushed
the
codex/issue-1517-gfql-coverage-audit
branch
2 times, most recently
from
May 18, 2026 22:13
d2379ef to
2d08851
Compare
lmeyerov
force-pushed
the
codex/issue-1517-gfql-coverage-audit
branch
from
May 18, 2026 22:19
2d08851 to
566c190
Compare
This was referenced May 18, 2026
perf: graphistry wheel size (625 kB) — GFQL engine files are large, investigate modularization
#1058
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #1517.
Follow-up: #1533 for repo-wide changed-line coverage hygiene.
Summary
bin/coverage_audit.py, a profile-based coverage.py audit helper that emits markdown + JSON zero/low-hit reports.gfql, for GFQL dead-code triage over parser, lowering, row-pipeline, temporal, AST, unified, and chain files.coverageandpytest-covto the normaltestextra so coverage infrastructure is managed with the rest of the test dependency stack.test-gfql-core (3.12)job viapytest-cov+ xdist instead of adding a duplicate standalone test run.gfql-coverage-audit-py3.12artifact and append the markdown report to the GitHub step summary.graphistry/tests/compute/gfql/coverage_baselines/ci-pandas-py3.12.json; new resolved GFQL target files must be added to the baseline.Validation
python3 -m py_compile bin/coverage_audit.pypython3 -m json.tool graphistry/tests/compute/gfql/coverage_baselines/ci-pandas-py3.12.jsonpython3 -m json.tool graphistry/tests/compute/gfql/coverage_baselines/rapids-25.02-cudf.jsonpython3 -m json.tool graphistry/tests/compute/gfql/coverage_baselines/rapids-26.02-cudf.jsonbash -n docker/test-cpu-entrypoint.sh docker/test-rapids-official-local.shpython3 -m pytest -q graphistry/tests/test_coverage_audit.py-> 8 passed./bin/ruff.sh bin/coverage_audit.py graphistry/tests/test_coverage_audit.py./bin/typecheck.sh bin/coverage_audit.py graphistry/tests/test_coverage_audit.pygit diff --check origin/master...HEAD2878 passed, 193 skipped, 15 xfailed; 42 target files; aggregate target coverage11984/15136 = 79.18%; baseline passed.391 passed, 17 skipped; audit consumed the generated.coveragefile.WITH_COVERAGE_AUDIT/COVERAGE_*envs:6209/15136 = 41.02%; baseline passed.6218/15136 = 41.08%; baseline passed. Known 25.02 warning:cuDriverGetVersion() takes no arguments; wrapper continued and tests passed.Coverage Stats
79.18%.41.02%.41.08%.Notes
Do not auto-merge.