Repository navigation
refactor: move rqe-optimizer to ASAPQuery - #232
Open
milindsrivastava1997 wants to merge 3 commits into
Open
milindsrivastava1997 wants to merge 3 commits into
milindsrivastava1997 wants to merge 3 commits into
Conversation
The RQE deployment optimizer is ASAPQuery's planner, so it moves there (ProjectASAP/ASAPQuery, rqe-optimizer/) with its history: the crate, its AutoSketch-vs-ASAP runner, the scripts that run, plot and summarize it, its committed results and its design docs (now rqe-optimizer/docs/). sketch-bench keeps the benchmarks and the analysis on top of them, including what turns measurements into solver input: the saturation study, the cost table, and the eval-table exporters (export_autosketch_eval_table.py, case_study_tables.py, export_case_studies.py, autosketch_benchmark_time.py). - The exporter test's committed tables move to scripts/testdata/, a copy of ASAPQuery's rqe-optimizer/results/autosketch-vs-asap-inputs/tables. end-of-file-fixer skips them: the test compares them byte for byte. - References to the moved docs point to ASAPQuery. - good_lp leaves the workspace; rqe-optimizer was its only user. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Removes what ProjectASAP/ASAPQuery#915 moves into ASAPQuery (
rqe-optimizer/, with its history):rqe-optimizercrate, its examples, tests, data and results;run_autosketch_vs_asap_synthetic.sh,extract_autosketch_plans.py,plot_autosketch_vs_asap_*.py,plot_sec63_paper.pyandsummarize_case_studies.py;docs/rqe_optimizer_*.md,rqe_sketch_deployment_v1.mdande2e_planning_flow.md.Merge after ASAPQuery#915.
What stays here
The benchmarks and the analysis on top of them, including what turns measurements into solver input:
study_saturation.pyexport_autosketch_eval_table.py,case_study_tables.py,export_case_studies.pyandautosketch_benchmark_time.pyOther changes
scripts/testdata/autosketch-vs-asap-tables/(100% renames). They're a copy of ASAPQuery'srqe-optimizer/results/autosketch-vs-asap-inputs/tables.end-of-file-fixernow skips them.good_lpleaves the workspace;rqe-optimizerwas its only user..gitignoreexceptions for the results' CSVs are gone.saturation_study.md, script comments) now point to ASAPQuery.ASAPQuery pins
aqpbm-coretoc750fa6, which stays reachable onmainafter this merges.Verified:
cargo clippy --workspace --all-targets --all-features --locked -D warnings,cargo fmt --check, andscripts/test_export_autosketch_eval_table.pypass.🤖 Generated with Claude Code