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

DINO encoder #22

Open
mli0603 opened this issue Mar 5, 2024 · 1 comment
Open

DINO encoder #22

mli0603 opened this issue Mar 5, 2024 · 1 comment

Comments

@mli0603
Copy link

mli0603 commented Mar 5, 2024

Dear authors:

Thank you for your great work! I wonder why you used DINOV1 instead DINOV2, which is more suitable for dense prediction task. Thank you!

@ZexinHe
Copy link
Collaborator

ZexinHe commented Mar 5, 2024

Hi,

Thanks for your interest!

We used DINOv1 during our first development to align with the original LRM paper and to avoid introducing new factors. Now that the codebase is been verified in v1.0, we switch to DINOv2 to ensure its absolute performance.

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

2 participants