Skip to content

Dedupe logic between forward and backward analysis pass - #1883

Open
linsword13 wants to merge 1 commit into
Ramble-Project:developfrom
linsword13:proj-cleanup
Open

linsword13 wants to merge 1 commit into
Ramble-Project:developfrom
linsword13:proj-cleanup

Conversation

@linsword13

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Lin Guo <linsword13@gmail.com>
@ramble-project-pr-bot

Copy link
Copy Markdown

Ramble Performance Test Metrics

Results produced with commit: 1995aee

Test Name Outcome Duration (s) Most Recent Run (s) Last 5 Avg (s)
test_analyze_large_file passed 1.2866 1.0816 (560ec6c) 1.0771
test_large_template_expansion passed 1.1030 1.1510 (560ec6c) 1.1216
test_many_experiments passed 17.9978 19.9880 (560ec6c) 18.1791
test_many_objects_defaults passed 11.5584 13.0167 (560ec6c) 11.5368
test_matrix_filter_perf passed 0.9683 1.0884 (560ec6c) 0.9604

@codecov

codecov Bot commented Oct 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.83333% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/ramble/ramble/analysis/base.py 95.09% 5 Missing ⚠️

📢 Thoughts on this report? Let us know!

This branch has not been deployed

No deployments
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