Skip to content
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

Show dropped image in output node #1150

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

bezo97
Copy link
Collaborator

@bezo97 bezo97 commented Oct 7, 2024

Closes #497 feature request.

Implementation details:

  • blob url and node id are optional parameters in loadGraphData
  • sets the preview image on all SaveImage nodes the original node it was saved by
  • using the output_node_id metadata when present
  • works for any dropped png or webp, also from the queue
  • PR does not include persistence of queue history

For reviewers:

  1. Drop an image that contains workflow metadata with a SaveImage node
  2. See dropped image in the node preview

- blobUrl is an optional paramtere in loadGraphData
- sets the preview image on SaveImage and PreviewImage nodes
- works for any dropped png or webp, also from the queue
@huchenlei
Copy link
Member

See #497 (comment)

This solution has a few missing parts:

  • Only handles image output
  • Only populte Preview Image and Save Image nodes
  • Blindly populates all Preview Image and Save Image nodes

- The file is displayed only on the node it was saved by.
- No longer restricted to specific supported node types
@bezo97
Copy link
Collaborator Author

bezo97 commented Oct 13, 2024

Thanks for your review @huchenlei. Here's a task list based on the discussed:

Persisting the queue history would be a way bigger task so I'm going with your 2nd suggestion ("single asset that has workflow as metadata back to correct output node")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants