cvg / LightGlue

LightGlue: Local Feature Matching at Light Speed (ICCV 2023)
Apache License 2.0
3.3k stars 317 forks source link

Homography pretraining shuffle? #77

Open ducha-aiki opened 10 months ago

ducha-aiki commented 10 months ago

Hi,

a bit of noob question - should I add some shuffling, or this is expected for homography pretraining? IMG_6165

Phil26AT commented 10 months ago

Hi,

yes this seems to be an issue here. You can fix it with "data.shuffle_training=True" in glue-factory. This might be an artifact from code refactoring, we will fix this.

hit2sjtu commented 10 months ago

Did not quite catch the issue here. Does this issue still exist in glue-factory codebase (since this is the LightGlue codebase). During my test running "data.shuffle_training=True" produces identical results as the default settings. @Phil26AT

aidongmandexiaowowo commented 3 weeks ago

Hi, did you train on the entire Homography dataset? I used a subset of the Homography dataset for training and the results were particularly poor