Skip to content

graph : refactor context to not pass gf explicitly #14629

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 18, 2025
Merged

graph : refactor context to not pass gf explicitly #14629

merged 1 commit into from
Jul 18, 2025
+295 −341

Conversation

ggerganov
Copy link
Member

target #14482

  • Keep the graph reference in llm_graph_context instead of passing it everywhere
  • Remove llm_graph_result_i abstraction

@ggerganov ggerganov mentioned this pull request Jul 11, 2025
Open
4 tasks
@ggerganov ggerganov force-pushed the gg/llama-reuse-graphs branch from 8303a68 to 3d28b3b Compare July 12, 2025 13:35
@ggerganov ggerganov force-pushed the gg/graph-context-refactor branch 2 times, most recently from 5a220cc to bc0a20c Compare July 12, 2025 19:51
@ggerganov ggerganov mentioned this pull request Jul 13, 2025
Merged
10 tasks
Base automatically changed from gg/llama-reuse-graphs to master July 17, 2025 16:08
@ggerganov ggerganov force-pushed the gg/graph-context-refactor branch from bc0a20c to 20010c4 Compare July 17, 2025 17:22
@ggerganov ggerganov force-pushed the gg/graph-context-refactor branch from 20010c4 to ae2fb57 Compare July 18, 2025 05:00
@ggerganov ggerganov merged commit 8f974bc into master Jul 18, 2025
56 of 57 checks passed
@ggerganov ggerganov deleted the gg/graph-context-refactor branch July 18, 2025 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ggerganov