Skip to content

Functional learned vtr for Warthog#226

Open
shemshesh wants to merge 6 commits intomainfrom
learned_VTR
Open

Functional learned vtr for Warthog#226
shemshesh wants to merge 6 commits intomainfrom
learned_VTR

Conversation

@shemshesh
Copy link
Contributor

No description provided.

@shemshesh shemshesh requested a review from a-krawciw September 23, 2024 21:05
Copy link
Contributor

@a-krawciw a-krawciw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Some clean up of comments to do.

  • Need to resolve how to use Git LFS for the model weights.

if (fully_matched && channel.cameras.size() == 2)
// return extractStereoFeaturesDisp(channel, channel_disp);
if (fully_matched && channel.cameras.size() == 2){
// CLOG(DEBUG, "stereo.learned_features") << "made it here";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove please.


return F::grid_sample(disparity, keypoints_norm, options).reshape({-1});
return output;
// return F::grid_sample(disparity, keypoints_norm, options).reshape({-1});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could drop to a single line again and remove the comment.

}
cv::Mat floatDisp;
disp.convertTo(floatDisp, CV_32F, 1.0f / disparity_multiplier);
// CLOG(DEBUG, "stereo.learned_features") << "disp_type " << disp.type();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove comments please.


// Get disparity for each keypoint
torch::Tensor disparity = getDisparityTensor(disp);

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove please.

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

Successfully merging this pull request may close these issues.

2 participants