-
Notifications
You must be signed in to change notification settings - Fork 156
feat(ai): add CASTS for GeaFlow reasoning ability #737
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Appointat
wants to merge
16
commits into
apache:master
Choose a base branch
from
Appointat:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
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
… Improved Decision Validation
…or structural signatures
…ss multiple files
…ality control Add native Gremlin simplePath() support to prevent pathological cycles in graph traversals. The implementation uses LLM-guided decision-making and AIMD confidence penalties rather than hard-coded restrictions, staying true to the system's learning philosophy. Key changes: - Add simplePath() step to Gremlin state machine for V, E, and P states - Implement per-request path history tracking in TraversalExecutor - Add cycle detection with configurable threshold and penalty modes - Enhance LLM Oracle prompts to recommend simplePath() for exploration goals - Add recent decision history context to improve LLM decision quality - Update configuration with CYCLE_PENALTY and CYCLE_DETECTION_THRESHOLD settings - Document design rationale and rejected alternatives in architecture.md - Add test case for simple path traversal validation
…ulations - Updated MetricsCollector to use Optional types for match_type, parent_node, parent_step_index, edge_label, sku_id, and decision parameters. - Enhanced EVALUATOR documentation to clarify evaluation phases and scoring mechanisms, including coverage rewards and penalties for cache misses. - Modified test cases in test_execution_lifecycle.py to align with new metrics structure and added tests for simple path execution. - Improved test coverage in test_gremlin_step_state_machine.py and test_lifecycle_integration.py to validate state transitions and integration with Gremlin state machine. - Refined threshold calculation tests to ensure monotonicity and boundary conditions. - Added dynamic execution environment constraints in documentation to clarify step legality in relation to current state and schema.
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.
Summary
How was this PR tested?