Skip to content

Fix: model reidentify losing path and failing on IP Adapters#8941

Merged
lstein merged 3 commits intoinvoke-ai:mainfrom
Pfannkuchensack:fix/reidentify-model-preserve-fields
Mar 7, 2026
Merged

Fix: model reidentify losing path and failing on IP Adapters#8941
lstein merged 3 commits intoinvoke-ai:mainfrom
Pfannkuchensack:fix/reidentify-model-preserve-fields

Conversation

@Pfannkuchensack
Copy link
Copy Markdown
Collaborator

Summary

The reidentify endpoint overwrote the model's relative path with an absolute path from the prober, and unconditionally accessed trigger_phrases which doesn't exist on all config types (e.g. IP Adapters), causing an AttributeError.

Related Issues / Discussions

Fixes #8940

QA Instructions

  1. Install an IP Adapter model (e.g. Standard Reference ViT-H) via Model Manager
  2. Reidentify the model
  3. Verify it no longer throws an AttributeError on trigger_phrases
  4. Verify the model path remains relative (not absolute)

Merge Plan

No special merge considerations.

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • ❗Changes to a redux slice have a corresponding migration
  • Documentation added / updated (if applicable)
  • Updated What's New copy (if doing a release after this PR)

The reidentify endpoint overwrote the model's relative path with an
absolute path from the prober, and unconditionally accessed
trigger_phrases which doesn't exist on all config types (e.g. IP
Adapters), causing an AttributeError.
@joshistoast
Copy link
Copy Markdown
Collaborator

Thanks for solving this so quick!

@lstein lstein self-assigned this Mar 7, 2026
@lstein lstein added the v6.12.0 Intended for 6.12.0 release label Mar 7, 2026
@github-actions github-actions Bot added api python PRs that change python files labels Mar 7, 2026
Copy link
Copy Markdown
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as advertised.

@lstein lstein enabled auto-merge (squash) March 7, 2026 01:16
@lstein lstein merged commit df225d3 into invoke-ai:main Mar 7, 2026
13 checks passed
@Pfannkuchensack Pfannkuchensack deleted the fix/reidentify-model-preserve-fields branch March 7, 2026 16:59
@lstein lstein moved this from 6.13.x to 6.12.x in Invoke - Community Roadmap Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api python PRs that change python files v6.12.0 Intended for 6.12.0 release

Projects

Status: 6.12.x

Development

Successfully merging this pull request may close these issues.

[bug]: Re-Identifying an IP Adapter results in error.

3 participants