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

Fix metadata format according to https://github.com/AUTOMATIC1111/sta… #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

@alexopus
Copy link

alexopus commented Jan 5, 2024

I merged this in my fork which is available in the Manager as "ComfyUI Image Saver". I renamed the nodes to avoid conflicts with this.

@receyuki
Copy link
Author

receyuki commented Jan 5, 2024

@alexopus In fact, I've created my own version of Saver based on the concept of this node and have added numerous new features.
https://github.com/receyuki/comfyui-prompt-reader-node
I believe you will be interested in this. Of course, if you have any suggestions or would like to contribute to this project please let me know, I would be very grateful.

@alexopus
Copy link

alexopus commented Jan 5, 2024

@receyuki Thanks for the hint, looks very nice indeed. Although perhaps it is missing the LoRAs/embeddings hashing for resource recognition on civitai? At least I couldn't find it in the readme. Anyway, I shall give that a try at some point.

@receyuki
Copy link
Author

receyuki commented Jan 5, 2024

@alexopus In fact, I am planning to add this feature in the next release, but the implementation will be somewhat different from your approach.

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.

Metadata format not cleanly compatible with A1111
2 participants