Skip to content

fix: honor configured graph model in cognify - #4633

Open
mikemikimike wants to merge 4 commits into
topoteretes:mainfrom
mikemikimike:fix/graph-model-config-4632
Open

fix: honor configured graph model in cognify#4633
mikemikimike wants to merge 4 commits into
topoteretes:mainfrom
mikemikimike:fix/graph-model-config-4632

Conversation

@mikemikimike

@mikemikimike mikemikimike commented Aug 23, 2026

Copy link
Copy Markdown

Fixes #4632

Root cause: config.set_graph_model() stored the model in GraphConfig, but cognify() and get_default_tasks() defaulted directly to KnowledgeGraph and never read it.

The fix resolves the configured model at call time when graph_model is omitted, preserves explicit graph_model precedence, and adds regression coverage at both the cognify forwarding layer and direct get_default_tasks() task construction layer.

Validation:

  • python -m py_compile passed for changed Python files
  • ruff check passed when available in the workspace
  • git diff --check passed
  • Full repository pytest was attempted but could not complete because upstream source downloads were repeatedly truncated by the network environment.

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.

[Bug]: cognee.config.set_graph_model() is inert — graph extraction ignores the configured model

1 participant