argmaxinc / DiffusionKit

On-device Inference of Diffusion Models for Apple Silicon
MIT License
495 stars 30 forks source link

Runtime error with argmaxtools #36

Open ngaloppo opened 1 month ago

ngaloppo commented 1 month ago

Submitting here, since I can't find the argmaxtools repo anymore, and I'm getting the error while using DiffusionKit.

Getting the following error after generation completes (but before the output image gets generated):

Traceback (most recent call last):
  File "/Users/ngaloppo/miniforge3/envs/diffusionkit/bin/diffusionkit-cli", line 33, in <module>
    sys.exit(load_entry_point('diffusionkit', 'console_scripts', 'diffusionkit-cli')())
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/ngaloppo/code/DiffusionKit/python/src/diffusionkit/mlx/scripts/generate_images.py", line 170, in cli
    image, _ = sd.generate_image(
               ^^^^^^^^^^^^^^^^^^
  File "/Users/ngaloppo/code/DiffusionKit/python/src/diffusionkit/mlx/__init__.py", line 512, in generate_image
    pprint(ic.os_spec())
           ^^^^^^^^^^^^
  File "/Users/ngaloppo/miniforge3/envs/diffusionkit/lib/python3.11/site-packages/argmaxtools/test_utils.py", line 383, in os_spec
    os_type, os_version, os_build_number = [
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: too many values to unpack (expected 3)
atiorh commented 1 month ago

Could you please share the output of sw_vers from your terminal?

ngaloppo commented 1 month ago

Unfortunately, I can't. What I can say is that it has more than three lines of output.

ZachNagengast commented 1 month ago

@ngaloppo Are the first three in the correct order as os_type, os_version, os_build_number? We can add a variable to accept the remaining values perhaps without crashing.

ngaloppo commented 1 month ago

Yes, that’s correct!

On Mon, Sep 23, 2024 at 6:11 PM Zach Nagengast @.***> wrote:

@ngaloppo https://github.com/ngaloppo Are the first three in the correct order as os_type, os_version, os_build_number? We can add a variable to accept the remaining values perhaps without crashing.

— Reply to this email directly, view it on GitHub https://github.com/argmaxinc/DiffusionKit/issues/36#issuecomment-2369905903, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABYVXCTFTAME4VL2IEAOOTZYC32TAVCNFSM6AAAAABOQRVAHOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRZHEYDKOJQGM . You are receiving this because you were mentioned.Message ID: @.***>