Skip to content

Conversation

@blessedcoolant
Copy link
Collaborator

@blessedcoolant blessedcoolant commented Sep 17, 2022

A bug fix patch for things that broke with the generate and dream refactor. #534

Fixes:

  • Fixed q not working to exit the dream command line.
  • Added `--sampler' to model_group too. This is an important argument that allows scripts to run a default sampler at boot.
  • Fixed generate ignoring the width and height from the config file.
  • Fixed some missing imports
  • Fixed some opts not ported over correctly.

ToDo:

  • CRITICAL >>> 2nd Prompt is Broken. The first prompt works. The 2nd prompt crashes. @lstein Please look into this.

This bug fix patch can be merged once all the bugs are fixed.

@lstein
Copy link
Collaborator

lstein commented Sep 17, 2022

I've got my build environment back (got some time on another university's cluster). I can't reproduce a crash on the second prompt. Do you mean that the first prompt of a session works and the second one fails? Could you paste in a screen grab or transcript?

I am very anxious to get things stable again, so will check back.

@blessedcoolant
Copy link
Collaborator Author

blessedcoolant commented Sep 17, 2022

I can't reproduce a crash on the second prompt. Do you mean that the first prompt of a session works and the second one fails?

Yes. Crashes for me 100% of the time.

Steps:

  1. Enter a prompt. Let it run.
  2. Press up and run same command again.
  3. Crash.

Here's the trace.

Traceback (most recent call last):
  File "scripts\dream.py", line 357, in <module>
    main()
  File "scripts\dream.py", line 95, in main
    main_loop(gen, opt, infile)
  File "scripts\dream.py", line 214, in main_loop
    file_writer = PngWriter(current_outdir)
  File "ldm\dream\pngwriter.py", line 20, in __init__
    os.makedirs(outdir, exist_ok=True)
  File "C:\Anaconda\envs\diffusion\lib\os.py", line 208, in makedirs
    head, tail = path.split(name)
  File "C:\Anaconda\envs\diffusion\lib\ntpath.py", line 185, in split
    p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not NoneType

@lstein lstein marked this pull request as ready for review September 17, 2022 03:08
Copy link
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for quickly finding and resolving these bugs. I am testing on my (new!) system now and will try to track down and fix the crashing you observed.

@lstein
Copy link
Collaborator

lstein commented Sep 17, 2022

I can't reproduce a crash on the second prompt. Do you mean that the first prompt of a session works and the second one fails?

Yes. Crashes for me 100% of the time.

Steps:

  1. Enter a prompt. Let it run.
  2. Press up and run same command again.
  3. Crash.

Here's the trace.

Traceback (most recent call last):
  File "scripts\dream.py", line 357, in <module>
    main()
  File "scripts\dream.py", line 95, in main
    main_loop(gen, opt, infile)
  File "scripts\dream.py", line 214, in main_loop
    file_writer = PngWriter(current_outdir)
  File "ldm\dream\pngwriter.py", line 20, in __init__
    os.makedirs(outdir, exist_ok=True)
  File "C:\Anaconda\envs\diffusion\lib\os.py", line 208, in makedirs
    head, tail = path.split(name)
  File "C:\Anaconda\envs\diffusion\lib\ntpath.py", line 185, in split
    p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not NoneType

Interesting. I'd found and fixed that one earlier today before I committed. I must not have fixed it correctly.

@blessedcoolant
Copy link
Collaborator Author

Thanks for quickly finding and resolving these bugs. I am testing on my (new!) system now and will try to track down and fix the crashing you observed.

Thanks. I'll continue the process of bug checking. If anything pops up, I'll pull up PR's or bring you in.

@lstein
Copy link
Collaborator

lstein commented Sep 17, 2022

Ok, I got the bug. Will fix one other minor thing I noticed and then commit.

lstein added a commit that referenced this pull request Sep 17, 2022
@lstein lstein merged commit 994c6b7 into invoke-ai:development Sep 17, 2022
@blessedcoolant blessedcoolant deleted the bug-fixes branch September 17, 2022 05:41
afiaka87 pushed a commit to afiaka87/lstein-stable-diffusion that referenced this pull request Sep 19, 2022
afiaka87 pushed a commit to afiaka87/lstein-stable-diffusion that referenced this pull request Sep 19, 2022
afiaka87 pushed a commit to afiaka87/lstein-stable-diffusion that referenced this pull request Sep 19, 2022
afiaka87 pushed a commit to afiaka87/lstein-stable-diffusion that referenced this pull request Sep 19, 2022
afiaka87 pushed a commit to afiaka87/lstein-stable-diffusion that referenced this pull request Sep 21, 2022
afiaka87 pushed a commit to afiaka87/lstein-stable-diffusion that referenced this pull request Sep 21, 2022
austinbrown34 pushed a commit to cognidesign/InvokeAI that referenced this pull request Dec 30, 2022
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.

2 participants