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

Update test-invoke-pip.yml #2524

Merged
merged 2 commits into from
Feb 6, 2023
Merged

Update test-invoke-pip.yml #2524

merged 2 commits into from
Feb 6, 2023

Conversation

mauwii
Copy link
Contributor

@mauwii mauwii commented Feb 5, 2023

test-invoke-pip.yml:

  • enable caching of pip dependencies in actions/setup-python@v4
  • add workflow_dispatch trigger
  • fix indentation in concurrency
  • set env PIP_USE_PEP517: '1'
  • cache python dependencies
  • remove models cache (since we currently use 190.96 GB of 10 GB while I am writing this)
  • add step to set INVOKEAI_OUTDIR
  • add outdir arg to invokeai
  • fix path in archive results

model_manager.py:

  • read files in chunks when calculating sha (windows runner is crashing otherwise)

@mauwii mauwii added the CI-CD Continuous integration / Continuous delivery label Feb 5, 2023
@mauwii mauwii requested a review from lstein February 5, 2023 06:21
@mauwii mauwii self-assigned this Feb 5, 2023
@mauwii mauwii force-pushed the update/ci/fix-test-invoke-pip branch 2 times, most recently from 89e06c9 to 166c063 Compare February 5, 2023 19:21
lstein
lstein approved these changes Feb 5, 2023
View reviewed changes
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.

I'm glad to see the cache disabled. It has been getting stuck very frequently recently.

@mauwii
update test-invoke-pip.yml
fc2670b 
- add workflow_dispatch trigger
- fix indentation in concurrency
- set env `PIP_USE_PEP517: '1'`
- cache python dependencies
- remove models cache (since currently 183.59 GB of 10 GB are Used)
- add step to set `INVOKEAI_OUTDIR`
- add outdir arg to invokeai
- fix path in archive results
@mauwii
update model_manager.py
a40bdef 
- read files in chunks when calculating sha
  - windows runner is crashing without
@mauwii mauwii force-pushed the update/ci/fix-test-invoke-pip branch from 2935244 to a40bdef Compare February 6, 2023 11:30
@mauwii mauwii enabled auto-merge February 6, 2023 11:30
@mauwii mauwii merged commit a485d45 into main Feb 6, 2023
@mauwii mauwii deleted the update/ci/fix-test-invoke-pip branch February 6, 2023 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lstein lstein lstein approved these changes

Assignees

@mauwii mauwii

Labels
CI-CD Continuous integration / Continuous delivery
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mauwii @lstein