Repository navigation
Submission: DeeptiTalesra - #35
Open
DeeptiTalesra wants to merge 8 commits into
Open
DeeptiTalesra wants to merge 8 commits into
DeeptiTalesra wants to merge 8 commits into
Conversation
Owner
|
Hey @DeeptiTalesra - thanks for the submission! Can you please remove the optimization to precompute source data in parallel? Solutions shouldn't modify the build harness. |
Contributor
✅ Results
Per-graph results
|
Use downstream critical-path weight instead of immediate work for scheduling and inline chaining decisions. Move queue puts outside the lock to avoid nested locking with SimpleQueue internals. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
_thread.join_thread is not available in the CI's Python 3.14t build. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Oversubscribe slightly (28 workers on 24 cores) so idle cores are filled when workers block on lock/queue. Reuse a single empty dict for root targets instead of allocating a new one per build call. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
✅ Results
Per-graph results
|
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
✅ Results
Per-graph results
|
Avoids repeated attribute lookups (dep.name, dep._idx) in the hot loop by precomputing sorted tuples at setup. Also makes build()'s internal sorted() see already-sorted keys — O(n) via Timsort. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
✅ Results
Per-graph results
|
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
✅ Results
Per-graph results
|
This branch has not been deployed
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.
Submission Checklist
submissions/DeeptiTalesra.py.Approach Description
Parallel build scheduler with source-data precomputation and inline chaining for Python 3.14t free-threading.
Key optimizations: