Repository navigation
feat(rust): opt-in in-process Rust graph queries for existing control room - #36
Closed
New1Direction wants to merge 14 commits into
Closed
New1Direction wants to merge 14 commits into
New1Direction wants to merge 14 commits into
Conversation
…alization failure
… real browser parity
…oof/performance limits
…raphIndex invalidation contract
Owner
Author
|
Closing during cleanup, as the owner asked. The subprocess bridge from #37 already gives opt-in Rust graph queries, and this branch's own benchmark shows the in-process path is slower (~4.3 ms vs ~0.27 ms in Python). The branch stays available if the indexed Generated by Claude Code |
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.
Experimental Rust PyO3 alternative — NOT a default backend
Important update: PR #37 was merged to
mainwhile this branch was in development, and already provides opt-in Rust-backed local HTTP graph queries via a bounded, credential-scrubbed subprocess bridge. Do not merge this PR as a competing runtime path without deciding which architecture to retain.This branch demonstrates a separate in-process Rust PyO3 adapter and an immutable native
GraphIndexthat can reuse a parsed graph across repeated view queries. The live Python HTTP server remains Python by default, and this branch is not deployed.Existing verified results (before latest GraphIndex change)
Latest unverified addition
GraphIndexclass for amortizing JSON parsing and reuse across multiple queries, with parity tests and build + per-query benchmark.Decision
Keep draft / do not merge until indexed native performance, cache invalidation semantics, and the overlap with merged PR #37 are resolved. This research should inform a single long-lived Rust graph runtime, not duplicate providers or game repository language percentages.
Lean 4 remains the verification layer. Bend 2 only on proven independent graph-kernel speedups.