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

SDAPI not parsing integers from strings sent in for parameters, length, width, seed, etc. #1427

Open
lenankamp opened this issue Mar 15, 2025 · 4 comments

Comments

@lenankamp
Copy link

Had a working demo using forge api, tried to drop in kobold for easier setup for a friend, no go. After much frustration trying to debug my side and looking for an error somewhere I finally just compared the network log of API requests. Totally my dumb fault for not sanitizing the variables going in. But if you want better parity with forge API, this would be nice. Or getting some error response would be helpful. As it is in the kobold console it says "Generate Image: The response could not be sent, maybe connection was terminated?", which is a bit misleading.

All working on my side now, so close if you think it's not worth your time, thanks for the great software.

@LostRuins
Copy link
Owner

May I know how does Forge handle the unsanitized variables? Or rather, which were the fields in question? The error might be causing the generation process to be aborted which is why you see it like this.

@lenankamp
Copy link
Author

My width, height, steps, and cfg_scale were all getting sent as strings, sloppy JavaScript. Just having them parse properly before sending corrected the problem, assume forge was doing that on the receiving side.

@LostRuins
Copy link
Owner

Ah, but does it work correctly with the real forge? Or does that error too

@lenankamp
Copy link
Author

Worked fine on forge, so never realized my slop til trying to setup friend with Kobold.

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

No branches or pull requests

2 participants