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

fix: Some more bugs on the lora MM #372

Merged
merged 13 commits into from
Dec 24, 2024
Merged

fix: Some more bugs on the lora MM #372

merged 13 commits into from
Dec 24, 2024

Conversation

db0
Copy link
Member

@db0 db0 commented Dec 12, 2024

Adds some more reporting on logs as well

@db0 db0 requested a review from tazlin December 12, 2024 16:03
@db0 db0 marked this pull request as draft December 14, 2024 11:31
@db0 db0 marked this pull request as ready for review December 23, 2024 14:05
@db0
Copy link
Member Author

db0 commented Dec 23, 2024

@tazlin take a look. I think I've caught everything. Running on my PC I validated both correct retaining of adhoc lora amount, recovering from corruption, and cleanup.

Copy link
Member

@tazlin tazlin left a comment

Choose a reason for hiding this comment

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

I don't see any red-flags on a quick review. I'm content to let this go to beta and see how it works in that setting.

@tazlin tazlin merged commit 4c106eb into main Dec 24, 2024
2 of 3 checks passed
@tazlin tazlin deleted the loraclean2 branch December 24, 2024 14:40
tazlin added a commit that referenced this pull request Dec 24, 2024
fix: correct bugs with adhoc loras and reference on disk (#372)
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