-
Notifications
You must be signed in to change notification settings - Fork 13
Adding a space after periods messes with (text:1.1) syntax #5
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
base: main
Are you sure you want to change the base?
Conversation
web/prompt_gallery.js
Outdated
return existing + ', ' + newText; | ||
} | ||
|
||
return existing + newText; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, cleaning the existing text can break the entire prompt. I hope this gets merged :)
If the original issue was to be sure to have a comma separating to new tags from the existing tags, maybe a fix can be:
return existing + newText; | |
return existing.split(",").concat(newText.split(",")).join(","); |
Thanks for submitting these, I've been working hard on the lora sidebar trying to get it into a good place and I'm almost there, I'll try to give these changes a review here in a day or two and merge things up! |
adding a space after period messes with weighting syntax (mouse:1.1) turns into (mouse:1. 1) which invalidates it