Skip to content

Add ARM NEON intrinsics to pointcloud - #13396

Merged
Nir-Az merged 1 commit into
realsenseai:developmentfrom
fateshelled:neon-pointcloud
Oct 13, 2024
Merged

Add ARM NEON intrinsics to pointcloud#13396
Nir-Az merged 1 commit into
realsenseai:developmentfrom
fateshelled:neon-pointcloud

Conversation

@fateshelled

Copy link
Copy Markdown
Contributor

Changes:

  • adding pointcloud_neon class
    • neon-pointcloud.cpp is based on the code in sse-pointcloud.cpp
    • tested on Ubuntu 22.04, OrangePi5 (RK3588s, 8GB of RAM) and RealSense D435.

I optimized pointcloud processing for ARM CPUs.
Please review it.

@sysrsbuild

Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@fateshelled fateshelled changed the title NEON support pointcloud Add ARM NEON intrinsics to pointcloud Oct 3, 2024
@@ -0,0 +1,242 @@
// License: Apache 2.0. See LICENSE file in root directory.
// Copyright(c) 2017 Intel Corporation. All Rights Reserved.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

2024 please

@Nir-Az

Nir-Az commented Oct 13, 2024

Copy link
Copy Markdown
Collaborator

@fateshelled thanks! Looks good..
Did you verify it works on the specific HW?
I will run some internal CI's in parallel to check for regressions..

@Nir-Az
Nir-Az merged commit ed6fc83 into realsenseai:development Oct 13, 2024
@fateshelled

Copy link
Copy Markdown
Contributor Author

Thank you for merging.
I checked that the point cloud looks correct in realsense-viewer. However, not verified that the values are consistent with other pointcloud classes.
Hardware used was Realsense D435 and OrangePi5.

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.

3 participants