Vespinian

Results 35 comments of Vespinian

Yes, it's an unfortunate side effect of initializing script_arg array to None so that I can then insert the args at the correct place. If the extension doesn't handle receiving...

@zero01101 Do you think you could test my tentative fix for this (#8669) to see if the error messages on your end go away while functionality remains? Seems to be...

@ljleb This fix should not have any effect on controlnet but if you have a bit of time to spare could you please test it out? Just running the controlnet...

Yes, it should be pretty simple to refit the scripts_args and indices, now that we are making a copy. Ideally, it would be another PR for this though, so we...

So I've cooked up something that resizes the arg list and updates the indices. Not sure if I should just push it to this PR. Anyway, while doing so I...

Parked the mentioned future arg refit in this [branch](https://github.com/Vespinian/stable-diffusion-webui/tree/feature-API-alwayson-scripts-resize-script_args) in case of misfortune. It's a 1 commit ahead branch from this repo but it should contain the fixes from this...

After a bit of fiddling, I have some code for that, if that is what you would prefer @AUTOMATIC1111. I can revert these and commit that instead. It just requires...

Then perhaps it would be better if this would be put in the `initialize_scripts` of the Scriptrunner class and have a `self.default_script_args` variable that I can just reuse for every...

So I added 2 list `default_script_arg_txt2img` and `default_script_arg_img2img` in the API class. They get initialized on the first request made by their respective tabs. From then on the `init_script_arg` function...

Changed to the proposed format and updated the PR description.