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

[Bug]: incorrect file path when prompt contains new line #4685

Closed
1 task done
inobrevi opened this issue Nov 13, 2022 · 2 comments · Fixed by #12327
Closed
1 task done

[Bug]: incorrect file path when prompt contains new line #4685

inobrevi opened this issue Nov 13, 2022 · 2 comments · Fixed by #12327
Labels
bug Report of a confirmed bug

Comments

@inobrevi
Copy link

Is there an existing issue for this?

  • I have searched the existing issues and checked the recent builds/commits

What happened?

prompt:

portrait of a woman, art by
Henry Asencio	
Anton Fadeev	
Artgerm
Bastien Lecouffe-Deharme	
Carne Griffiths	
Darek Zabrocki	
Patrice Murciano	
Russ Mills	
WLOP
Yoji Shinkawa	

error:

OSError: [Errno 22] Invalid argument: 'outputs/txt2img-images\\02357-757535083-portrait of a woman, art by_Henry Asencio\t_Anton Fadeev\t_Artgerm_Bastien Lecouffe-Deharme\t_Carne Griffiths\t_Darek Zabrocki\t_Patr.png'

Steps to reproduce the problem

use prompt with new line symbol

What should have happened?

The script should convert that to safe saving reading name

Commit where the problem happens

98947d1

What platforms do you use to access UI ?

Windows

What browsers do you use to access the UI ?

Google Chrome

Command Line Arguments

No response

Additional information, context and logs

No response

@inobrevi inobrevi added the bug-report Report of a bug, yet to be confirmed label Nov 13, 2022
@krisfail
Copy link
Collaborator

Could not reproduce. More information would be appreciated.

@krisfail krisfail added the cannot-reproduce I can't reproduce this, so I can't fix it. Add steps for reproduction and remove this tag. label Nov 16, 2022
@inobrevi
Copy link
Author

inobrevi commented Nov 16, 2022

Thank you for looking into it.
With new insight: it is not new line, it is '\t' tab symbol.
Basically possible only when pasting.

@mezotaken mezotaken added bug Report of a confirmed bug upstream Issue or feature that must be resolved upstream and removed cannot-reproduce I can't reproduce this, so I can't fix it. Add steps for reproduction and remove this tag. bug-report Report of a bug, yet to be confirmed labels Jan 22, 2023
@catboxanon catboxanon removed the upstream Issue or feature that must be resolved upstream label Aug 3, 2023
@catboxanon catboxanon linked a pull request Aug 5, 2023 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Report of a confirmed bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants