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

Uncleaned LoRA components cause errors during training #70

Closed
ShenhanQian opened this issue Feb 25, 2025 · 3 comments
Closed

Uncleaned LoRA components cause errors during training #70

ShenhanQian opened this issue Feb 25, 2025 · 3 comments

Comments

@ShenhanQian
Copy link
Contributor

Hi, thanks for sharing the complete code base!

When I tried to train the model, the following two places caused errors because either args.load_lora or args.lora is undefined for MonSt3R:

https://github.com/Junyi42/croco/blob/6f1362bcd0a76821ca3091dc2d0c3253e95c7794/utils/misc.py#L307-L310

https://github.com/Junyi42/croco/blob/6f1362bcd0a76821ca3091dc2d0c3253e95c7794/utils/misc.py#L341-L344

@lifuguan
Copy link

Same problem here. Additionally, it seems the paper doesn’t mention any LoRA configurations or training details. I’m curious to know more about that.

@Junyi42
Copy link
Owner

Junyi42 commented Mar 20, 2025

Hi @ShenhanQian @lifuguan,

Thanks for pointing out the issue. The LORA configuration is an experimental feature and not applied in the paper. We have updated the code in croco to fix this issue: Junyi42/croco#1.

Best

@ShenhanQian
Copy link
Contributor Author

Thanks for the update.

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

No branches or pull requests

3 participants