Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add tanks and temples dataset loader #653
Add tanks and temples dataset loader #653
Changes from 27 commits
392f8b1
0977efb
4d65e30
5cfb8fc
a4b8775
77b9178
7038d18
117fdc6
324fc98
432daa9
581c6cb
d274689
346a7fe
e8efe4d
40418ec
1aadc03
888246d
7b80409
6efbeb0
af1d26e
8fe66bf
be50f6e
888f878
8d3b339
b6fdffb
b3a652c
1e5b58e
52b1140
e0186d0
1703165
cb31830
00c5d4f
6687f8a
5824974
c70e937
750e147
d4802dc
cbb68c5
9062f12
52ee748
968d16a
1ba2dc2
6faf44a
c7864be
98c8d0c
39b3cf8
5554638
6a331d4
25c0f68
2f3e9e2
574ac81
38fbe1d
f81667e
897923a
ef74582
f59d10c
bf4cbbf
eadd6ee
dd3d3c9
97304d3
68a8fa1
96de130
51ed829
cfb46fe
f366590
b3d0d9e
46dd6f9
9b26318
6fabe5d
8ce2a4c
78e4e1d
62c99d2
e658341
279623e
420cfb4
8aa9f9f
d6ebb47
ea235cd
afd83e2
8fd61c1
9b0be98
59bf16b
44f02f7
6b5dc0e
92fc731
614f682
aa9618c
793e51c
09bc83a
7254c29
39504fe
ee861ec
9d7596d
c3fab34
ed69f72
8d6cf88
640ad75
6021108
b7d9dc3
5778d1f
d0c9e67
5529914
a60732d
38971bd
f781d32
eb2dc0c
29cc2c3
cb801a3
ee6ba95
0cbee24
8f5a78b
aa75137
e005be2
634f220
75d92a6
511c361
68f75aa
78d7282
0a74180
7fb8630
6d3ca41
ea532a9
2e4eda5
34a0358
f3fd910
480a9f9
0043d37
a429482
706ef7e
783344b
4f9005c
c084bd1
845f35c
e032f49
0237101
303b695
e564993
c79c21c
306893b
5c9e26d
c2a3ee6
3ca4559
6e6b691
032606a
5448dee
a8b6532
bc16d0d
7195956
6e97ff6
4abf136
395ff3e
f3f8926
c3a9b57
4afbfa5
4850f1f
6ae33ae
c0a14a7
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am wondering if we can just generate the matches apriori and use something like a cache to just load the matches from the disk.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was also thinking this would be the best approach, as this is more for debugging and will likely only be used by us.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see -- if we wish to use the Tanks and Temples loader to get GT poses to measure pose error, what's the difference between computing matches offline vs. online?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because the synthetic matching frontend is only used for debugging, and I'm not sure we should merge it into the main repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ayushbaid @travisdriver wanted to revisit this -- I would prefer to keep as-is and make this code self-contained so that we only need to run one command, instead of having to make two new scripts (one to generate all the correspondences beforehand, and then making a new one to accept saved correspondences)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will have to be pinned to the machine with the input worker, right? Hence I was suggesting we create the correspondences on disk beforehand. But fine with it given the time constraint.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could just avoid futures for this file I guess?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm wouldn't we lose all parallelization benefits then?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but since we are just loading from disk, we would not lose much in terms of time I think