3DTopia / OpenLRM

An open-source impl. of Large Reconstruction Models
Apache License 2.0
966 stars 56 forks source link

DINO encoder #22

Open mli0603 opened 8 months ago

mli0603 commented 8 months ago

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 commented 8 months ago

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.