TemryL / ComfyS3

ComfyS3 seamlessly integrates with Amazon S3 in ComfyUI. This open-source project provides custom nodes for effortless loading and saving of images, videos, and checkpoint models directly from S3 buckets within the ComfyUI graph interface.
GNU General Public License v3.0
30 stars 17 forks source link

Output directory incorrect #10

Closed ckao10301 closed 3 months ago

ckao10301 commented 3 months ago

@TemryL In the .env, I used "output" as the output directory.

But when the file is saved, it's not saved to the "output" folder. Instead "output\" is added as a prefix to the file name. What am I doing wrong?

image image

ppvolto commented 3 months ago

Do you use Windows or the Windows Subsystem for Linux?

ckao10301 commented 3 months ago

Windows comfy portable edition.

On Sat, Jun 15, 2024 at 12:12 AM ppvolto @.***> wrote:

Do you use Windows or the Windows Subsystem for Linux?

— Reply to this email directly, view it on GitHub https://github.com/TemryL/ComfyS3/issues/10#issuecomment-2169173383, or unsubscribe https://github.com/notifications/unsubscribe-auth/A6462DRV37LEGWPIC2OLTO3ZHPSOPAVCNFSM6AAAAABJLEX3HCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRZGE3TGMZYGM . You are receiving this because you authored the thread.Message ID: @.***>

ppvolto commented 3 months ago

The Problem is that Windows uses the \ as path separator and urls and Linux use the / for that task. So that is a Bug in the filename generation

ckao10301 commented 3 months ago

The Problem is that Windows uses the \ as path separator and urls and Linux use the / for that task. So that is a Bug in the filename generation

thanks, I decided to work in linux instead.