Skip to content

Conversation

Hapseleg
Copy link

adding a space after period messes with weighting syntax (mouse:1.1) turns into (mouse:1. 1) which invalidates it

return existing + ', ' + newText;
}

return existing + newText;
Copy link

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:

Suggested change
return existing + newText;
return existing.split(",").concat(newText.split(",")).join(",");

@Kinglord
Copy link
Owner

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!

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.

3 participants