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 --log_tokenization to sysargs #2523

Merged
merged 8 commits into from
Feb 5, 2023
Merged

Add --log_tokenization to sysargs #2523

merged 8 commits into from
Feb 5, 2023

Conversation

whosawhatsis
Copy link
Contributor

This allows the --log_tokenization option to be used as a command line argument (or from invokeai.init), making it possible to view tokenization information in the terminal when using the web interface.

This allows the --log_tokenization option to be used as a command line argument (or from invokeai.init), making it possible to view tokenization information in the terminal when using the web interface.
@blessedcoolant
Copy link
Collaborator

This seems fine to me. And it's a handy feature to have it on always for the WebUI.

I also added a few more things to this PR.

  • Removed the deprecation warning that was being thrown in condition for the frontend user. The user won't care about this warning and its just clutter.
  • Cleaned up and beautifed a bunch of logging statements so the WebUI's console feels cleaner and easier to read.
  • Tagged as Token Logs accordingly.

@blessedcoolant
Copy link
Collaborator

blessedcoolant commented Feb 5, 2023

@lstein Please give a second review on this PR. I don't want to be the solo reviewer because I added some stuff to this PR too. Enable auto merge if you think it's all good to go.

Copy link
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me; worked fine on test.

@lstein lstein enabled auto-merge February 5, 2023 15:33
@lstein lstein merged commit 4ada4c9 into invoke-ai:main Feb 5, 2023
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.

None yet

3 participants