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

Runtime error with argmaxtools #36

Open
ngaloppo opened this issue Sep 19, 2024 · 4 comments
Open

Runtime error with argmaxtools #36

ngaloppo opened this issue Sep 19, 2024 · 4 comments

Comments

@ngaloppo
Copy link

ngaloppo commented Sep 19, 2024

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
Copy link
Contributor

atiorh commented Sep 19, 2024

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

@ngaloppo
Copy link
Author

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

@ZachNagengast
Copy link
Collaborator

@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
Copy link
Author

ngaloppo commented Sep 24, 2024 via email

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

3 participants