-
Notifications
You must be signed in to change notification settings - Fork 74
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
KeyError: 'ref_junction_map' #83
Comments
Hi, in your config file |
@rpautrat I am sorry but there are so many terminologies in SOLDv2 work which is making it complex to adapt. what is difference in "official" and "exported" dataset ? Can I directly jump to step5 for training on my custom data ? step 1 is training model on synthetic which is not needed to do again because we already have trained model by you guys, am I correct ? |
You are right that Step 1 is not necessary if you use the pre-trained model. The Wireframe dataset does have existing labels, this is what you would use if you would select the option gt_source_train= 'official'. But the point of SOLD2 is to not use these labels (which are biased towards wireframe lines only), and we instead propose to export a pseudo ground truth that does not have this bias. That's why Steps 2 and 3 are necessary. I hope this is clearer now. |
I still get the same error when I run step 5 directly with gt_source_train="official" and gt_source_test="official" and also return_type: "paired_desc" |
When I try step 4 I get the following error -
I used the command -
When I try step 2 I get the following error -
I used the command - |
Now that I remember, the code is actually not made to train on the official ground truth. We only used it to evaluate the line descriptors on these lines, but it is currently not compatible with training. What you could do for step 4 would be to modify the return function of the wireframe dataloader SOLD2/sold2/dataset/wireframe_dataset.py Lines 548 to 559 in 3d7bcd6
If you modify the keys 'line_map_pos' to 'line_map' and 'heatmap_pos' to 'heatmap', it might work. Step 5 with the 'paired_desc' option is not available at all currently. It would need additional changes to the dataloader. But again, the goal of SOLD2 is to train without existing ground truth, so it doesn't make sense to use the official ground truth, which is not good for localization for example. |
Regarding the error "TypeError: warp_perspective() got an unexpected keyword argument 'flags'", the issue is with your kornia version. We used an older one that had different arguments. Please consider using kornia==0.3.0 and it should solve it. |
I updated kornia==0.3.0 and Now it installed torch==1.5.0. Now its giving me error -
Now I Installed |
@rpautrat Where do I give path to my custom data ? |
Training on your own data is not supported for now, but several people did it by creating a new dataloader inspired by holicity_dataset.py. More details in this issue: #19 (comment) |
I get the following error when I try to train on wireframe dataset.
Wireframe doesn't have this key.
python -m sold2.experiment --mode train --dataset_config sold2/config/wireframe_dataset.yaml --model_config sold2/config/train_full_pipeline.yaml --exp_name sold2_full_v0.5.7 --pretrained --pretrained_path pretrained_model/ --checkpoint_name sold2_wireframe.tar
The text was updated successfully, but these errors were encountered: