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

Add main.log upload steps to issue template #572

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

webfiltered
Copy link
Contributor

@webfiltered webfiltered commented Dec 26, 2024

Proposed:

image

┆Issue is synchronized with this Notion page by Unito

@webfiltered webfiltered requested a review from a team as a code owner December 26, 2024 23:31
@robinjhuang
Copy link
Member

Should we include comfyui.log as well?

@webfiltered
Copy link
Contributor Author

My thought here was just to cover startup issues (before comfyui.log is even opened) - but you're right, if we do need it for any reason, the delay for the user would likely be worse than just uploading two logs.

@ltdrdata
Copy link
Member

Should we include comfyui.log as well?

For reference, the comfyui.log is set to be generated by ComfyUI-Manager, and in the feat/cnr branch, the save path has been moved to the user directory.

@webfiltered
Copy link
Contributor Author

comfyui.log is set to be generated by ComfyUI-Manager

The desktop app logs python server output to comfyui.log in:

# macOS:
~/Library/Logs/ComfyUI

# Windows:
%APPDATA%\ComfyUI\logs

So long as Manager is writing in another directory, all should be well. 😄

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

Successfully merging this pull request may close these issues.

3 participants