Skip to content

optimize graph analysis - #490

Draft
woutdenolf wants to merge 1 commit into
mainfrom
40-optimize-graph-analysis
Draft

woutdenolf wants to merge 1 commit into
mainfrom
40-optimize-graph-analysis

Conversation

@woutdenolf

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Aug 5, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.91120% with 16 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/ewokscore/graph/taskgraph.py 80.00% 10 Missing ⚠️
src/ewokscore/graph/analysis.py 0.00% 2 Missing ⚠️
src/ewokscore/graph/serialize.py 33.33% 2 Missing ⚠️
src/ewokscore/graph/_freeze.py 97.50% 1 Missing ⚠️
src/ewokscore/graph/error_handlers.py 95.65% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@woutdenolf
woutdenolf force-pushed the 40-optimize-graph-analysis branch 7 times, most recently from 19b4659 to 232af16 Compare August 11, 2026 09:27
@woutdenolf woutdenolf added this to the v6.0 milestone Sep 8, 2026
@abmajith

Copy link
Copy Markdown
Member

@woutdenolf

could you provide some context in what way you try to optimize the graph analysis, if I know the context, I can test, and report the difference, good thing for me to learn in this aspect.

@woutdenolf

Copy link
Copy Markdown
Member Author

This is done but I want to wait until 5.1 is released.

The biggest speed-up was not going through the same links multiple times but caching things (hence the GraphAnalysis class instead of functions).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@woutdenolf
woutdenolf force-pushed the 40-optimize-graph-analysis branch from 8667e54 to a3b89d8 Compare October 24, 2026 10:05

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.

Optimize graph analysis

2 participants