Skip to content

Conversation

@blessedcoolant
Copy link
Collaborator

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

  • Refactor
  • Feature
  • Bug Fix
  • [?] Optimization

Have you discussed this change with the InvokeAI team?

  • No

Description

  • Fixed filter type select using images instead of all -- Probably some merge conflict.
  • Added loading state for model lists. Handy when the model list takes longer than a second for any reason to fetch. Better to show this than an empty screen.
  • Refactored the render model list function so we modify the display component in one area rather than have repeated code.

Other Issues

  • The list can get a bit laggy on initial load when you have a hundreds of models / loras. This needs to be fixed. Will make another PR for this.

Copy link
Contributor

@psychedelicious psychedelicious left a comment

Choose a reason for hiding this comment

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

I fixed a number of conflicts from the Onnx merge

@blessedcoolant blessedcoolant merged commit c9d452b into invoke-ai:main Aug 1, 2023
@blessedcoolant blessedcoolant deleted the mm-ui-fixes branch August 1, 2023 13:06
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.

2 participants