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

fixes in SDLoraManager #212

Merged
merged 3 commits into from
Jan 29, 2024
Merged

fixes in SDLoraManager #212

merged 3 commits into from
Jan 29, 2024

Conversation

catwell
Copy link
Member

@catwell catwell commented Jan 26, 2024

No description provided.

@catwell catwell force-pushed the pr/sd-lora-manager-fixes branch from 2bd9eeb to edd83ea Compare January 26, 2024 16:06
@limiteinductive limiteinductive self-requested a review January 26, 2024 16:06
deltheil
deltheil previously approved these changes Jan 26, 2024
@catwell catwell marked this pull request as draft January 26, 2024 17:32
@catwell
Copy link
Member Author

catwell commented Jan 26, 2024

As of now tests don't pass.

@catwell catwell dismissed stale reviews from deltheil and limiteinductive via 8acb369 January 26, 2024 18:30
@catwell catwell force-pushed the pr/sd-lora-manager-fixes branch from e7f7e84 to 8acb369 Compare January 26, 2024 18:30
@catwell catwell marked this pull request as ready for review January 26, 2024 18:30
- support _out_0
- sort _in before _out
- avoid false positives by only considering suffixes
@catwell catwell force-pushed the pr/sd-lora-manager-fixes branch from 8acb369 to dcd50d1 Compare January 26, 2024 18:32
(also simplify list comprehension for exclusion list)
@catwell catwell merged commit deb5e93 into main Jan 29, 2024
1 check passed
@catwell catwell deleted the pr/sd-lora-manager-fixes branch January 29, 2024 10:11
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.

3 participants