griptape-ai / ComfyUI-Griptape

A suite of Griptape nodes for ComfyUI
Apache License 2.0
45 stars 4 forks source link

can't use combine text to set a lora name string widget. #44

Closed spiraloid closed 3 days ago

spiraloid commented 4 days ago

image

shhlife commented 4 days ago

Thanks @spiraloid - taking a look!

It looks like the issue may be that the input type for lora_name is an item from a list..

Here's how it's defined in the class:

    @classmethod
    def INPUT_TYPES(s):
        return {"required": { "model": ("MODEL",),
                              "clip": ("CLIP", ),
                              "lora_name": (folder_paths.get_filename_list("loras"), ),
                              "strength_model": ("FLOAT", {"default": 1.0, "min": -100.0, "max": 100.0, "step": 0.01}),
                              "strength_clip": ("FLOAT", {"default": 1.0, "min": -100.0, "max": 100.0, "step": 0.01}),
                              }}

I'll have to dig around and see if I can figure out what type of value it takes.

Workin on it! :)

shhlife commented 4 days ago

@spiraloid Okay, I've just pushed a new node that will convert a string to a combo input type..

image

Can you pull the latest and give it a try?

Thanks! :)

spiraloid commented 3 days ago

awwww yeah