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.

Sign up for GitHub

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

Jump to bottom

Installer tweaks #4070

Merged
merged 5 commits into from
Jul 30, 2023
Merged

Installer tweaks #4070

merged 5 commits into from
Jul 30, 2023

Conversation

lstein
Copy link
Collaborator

@lstein lstein commented Jul 29, 2023 •
edited
Loading

What type of PR is this? (check all applicable)

  • [ X] Optimization

Have you discussed this change with the InvokeAI team?

  • [X ] Yes
  • No, because:

Have you updated all relevant documentation?

  • [X ] Yes
  • No

Description

This PR does two things:

  1. if the environment variable INVOKEAI_ROOT is defined at install time, the zipfile installer will default to its value when asking the user where to install the software
  2. If the user has more than 72 models of any type installed, then the list will be truncated in the TUI and the user given a warning. Anything larger than this number of models causes the vertical space to overflow. The only effect of truncation is that the user will not be able to see and delete the models that were truncated. The message advises the user to go to the Web Model Manager interface in this event.

closes #4046

@lstein lstein requested a review from ebr as a code owner July 29, 2023 23:23
@lstein lstein merged commit adfcb61 into main Jul 30, 2023
7 checks passed
@lstein lstein deleted the install/tui-tweaks branch July 30, 2023 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hipsterusername hipsterusername hipsterusername approved these changes

@ebr ebr Awaiting requested review from ebr ebr is a code owner

@JPPhoto JPPhoto Awaiting requested review from JPPhoto

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bug]: invokeai installs to the wrong directory
2 participants
@lstein @hipsterusername