Skip to content

Conversation

@psychedelicious
Copy link
Contributor

@psychedelicious psychedelicious commented Aug 1, 2023

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Community Node Submission

Have you discussed this change with the InvokeAI team?

  • Yes
  • No, because:

Have you updated all relevant documentation?

  • Yes
  • No

Description

Note: this PR is based on #4105

fix(db): retrieve metadata even when no session_id

this was unnecessarily skipped if there was no session_id.

Related Tickets & Documents

https://discord.com/channels/1020123559063990373/1134237957381947563/1135810973446439033

QA Instructions, Screenshots, Recordings

Metadata should load for images without a session. This is a fix for @techjedi's import script (on discord).

Added/updated tests?

  • Yes
  • No : nyet

[optional] Are there any post deployment tasks we need to perform?

nah

Copy link
Contributor

@maryhipp maryhipp left a comment

Choose a reason for hiding this comment

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

The metadata change looks good to me - not sure about the onnx stuff

this was unnecessarily skipped if there was no `session_id`.
@psychedelicious psychedelicious force-pushed the fix/db/metadata-no-session branch from 957e551 to 9a8b1c6 Compare August 1, 2023 13:54
@psychedelicious
Copy link
Contributor Author

@maryhipp Sorry I had based this on a diff branch, fixed

Copy link
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.

LGTM

Copy link
Contributor

@brandonrising brandonrising left a comment

Choose a reason for hiding this comment

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

lgtm

@psychedelicious psychedelicious merged commit a6f9396 into main Aug 3, 2023
@psychedelicious psychedelicious deleted the fix/db/metadata-no-session branch August 3, 2023 00:43
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.

5 participants