StoryDiffusion icon indicating copy to clipboard operation
StoryDiffusion copied to clipboard

add.py modified to have one core app.

Open lvalics opened this issue 1 year ago • 4 comments

I modified the app.py file because I saw that it was no longer used (probably it was an older code). Now, having three versions—low, normal, and Mac—will create chaos in the code for maintenance. My suggestion is to review the app.py and see if this approach is okay.

At the moment, for some reason, it generates images with noise; I probably made a mistake, but someone who knows the code could surely find the error (if I don’t have time to find it myself).

It starts with a command line, for example:

python app.py --mode normal --debug

It’s a draft, do not make it public until the bug is fixed and tested.

lvalics avatar May 08 '24 12:05 lvalics

Now it is working, if you approve, I will add latest modification you added ... Also one part is not clear to me. Check here, does not get global, so I cannot separate logic. print ("Low VRAM 2")

lvalics avatar May 08 '24 19:05 lvalics

If you want to drop main one, and keep only the low vram, just reject this modification, it is not needed then.

lvalics avatar May 08 '24 20:05 lvalics

I modified the app.py file because I saw that it was no longer used (probably it was an older code). Now, having three versions—low, normal, and Mac—will create chaos in the code for maintenance. My suggestion is to review the app.py and see if this approach is okay.

I can‘t agree more, current code is chaos. It appears that the original version can be deleted, with the low VRAM version and the Apple version being retained. Also app.py maybe too long, Perhaps some of the code needs to be moved into the utils. I am currently developing a Comfui plugin and will come back to organize the code later.

Z-YuPeng avatar May 09 '24 06:05 Z-YuPeng

OK, so for now drop this one.

lvalics avatar May 09 '24 06:05 lvalics