-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Integrate sd-v1-5 model into test matrix (easily expandable), remove unecesarry caches #1293
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
Merged
lstein
merged 35 commits into
invoke-ai:development
from
mauwii:integrate-models-into-test-matrix
Oct 30, 2022
Merged
Integrate sd-v1-5 model into test matrix (easily expandable), remove unecesarry caches #1293
lstein
merged 35 commits into
invoke-ai:development
from
mauwii:integrate-models-into-test-matrix
Oct 30, 2022
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
mauwii
commented
Oct 29, 2022
- integrated the sd-v-1-5 mdoel into the test matrix, which is easyli expandable / changeable for future changes
- switched to macOS-12 since latest still points to "Using 12 soon" and we are now at 13...
- removed unecesarry caches from test-invoke-conda after realizing that it was over 30GB / 10GB offered for a free account 🙈
- made build-container.yml's cache reusable, which also heavily speeds up container creation
- before every build created a new cache....
- renamed test-invoke-conda workflow by removing the redundant parts from name
instead of --user username:token
use conda packages where possible as mentioned in conda docs https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/manage-pkgs.html#installing-non-conda-packages
also loosen verisons of scipy flask-socketio flask_cors
also directly setup correct conda env
- also update sd-cache display name to include current model
since an upcoming update makes it interactive
... on top of conda env activation, since `~/.cache` also contains pip
since free 10GB Limit is already overused multiple times
- pull_request_target trigger does not verify the requesters commit(s) - is more to be used for automations like labeling, commenting, ... - stuff where a token with write access to the repo is necesarry
lstein
approved these changes
Oct 30, 2022
Collaborator
lstein
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm impressed with how rapidly you're improving the workflows. Keep 'em coming.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.