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

About find_scale_from_depth inputs #14

Open
DanRos22 opened this issue Oct 1, 2020 · 1 comment
Open

About find_scale_from_depth inputs #14

DanRos22 opened this issue Oct 1, 2020 · 1 comment

Comments

@DanRos22
Copy link

DanRos22 commented Oct 1, 2020

DF-VO/vo_modules.py

Lines 766 to 768 in e9d91ba

scale = self.find_scale_from_depth(
cur_data[self.cfg.translation_scale.kp_src], ref_data[self.cfg.translation_scale.kp_src][ref_id],
E_pose.inv_pose, self.cur_data['depth']

Why the depth map is taken from current sample (self.cur_data['depth']), and not taken from reference?
According to function definition and implementation in
def find_scale_from_depth(self, kp1, kp2, T_21, depth2):
the depth should correspond to kp2 (which is ref), and not to kp1 (cur).
What am I missing?
Thanks for this work!

@Huangying-Zhan
Copy link
Owner

Hi @DanRos22 , sorry for late reply.
Please refer to the previous issue #7 .
Simple explanation will be:
cur_data[self.cfg.translation_scale.kp_src] actually stores kp_ref
while
ref_data[self.cfg.translation_scale.kp_src][ref_id] stores kp_cur.
The reason is mentioned in #7

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