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

[depth_image_proc/register] option to fill sparsity holes on upsampling #363

Merged
merged 1 commit into from
Nov 4, 2019

Conversation

ipa-rmb
Copy link

@ipa-rmb ipa-rmb commented Nov 3, 2018

Added the option to fill the sparse areas with nearest neighbor depth values on upsampling operations in depth_image_proc/register . We make use of this function quite frequently. The PR does not alter the standard behavior.

…lues on upsampling operations in depth_image_proc/register
@ipa-rmb
Copy link
Author

ipa-rmb commented Jan 21, 2019

Hello, could any maintainer please comment or merge this PR? Many thanks!

Copy link
Member

@SteveMacenski SteveMacenski left a comment

Choose a reason for hiding this comment

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

I'm OK with this -- @JWhitleyAStuff ?

@JWhitleyWork
Copy link
Collaborator

I've got some formatting and C++-y complaints but nothing technical. They can be fixed when I lint the ROS1 side in the near future.

@SteveMacenski SteveMacenski merged commit a24ac8b into ros-perception:indigo Nov 4, 2019
@JWhitleyWork
Copy link
Collaborator

@SteveMacenski - I'm guessing you didn't intend for this to target indigo?

@SteveMacenski
Copy link
Member

That would be correct. I’ll add to my queue to revert and retarget to melodic

abhijitmajumdar added a commit to abhijitmajumdar/image_pipeline that referenced this pull request Feb 17, 2021
schornakj added a commit to schornakj/image_pipeline that referenced this pull request Aug 20, 2021
Signed-off-by: Joe Schornak <joe.schornak@gmail.com>
clalancette pushed a commit to schornakj/image_pipeline that referenced this pull request Apr 26, 2022
Signed-off-by: Joe Schornak <joe.schornak@gmail.com>
clalancette added a commit that referenced this pull request Apr 26, 2022
* port upsampling interpolation from #363

Signed-off-by: Joe Schornak <joe.schornak@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@openrobotics.org>
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.

None yet

3 participants