PointCloudLibrary / PointCloudLibrary/pcl

[sample_consensus] Migrated the SSE and AVX implementations of countWithinDistance to RISC-V RVV 1.0

Open
#6,385 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

kind: request status: triage
Dominant language
C++
Stars
11.1k
Forks
4.7k
Avg merge
4d 10h
Merged PRs (30d)
6

Description

Hello everyone,

I've recently been working on some CPU architecture-related contributions for PCL. Referring to https://github.com/PointCloudLibrary/pcl/pull/3519, I have migrated the SSE and AVX implementations of countWithinDistance to RISC-V RVV 1.0, implementing countWithinDistanceRVV. The code repository is available at:

https://github.com/CelestialMelody/pcl/tree/feature/rvv10

The submitted content includes the following:

  1. Chinese documentation recording the environment setup, compilation, implementation, and testing processes. These documents are not intended for merging into PCL (I can provide English translations if needed):
    RISCV Development Environment Setup
    PCL Cross-Compilation Guide for RISC-V
    RVV Implementation Supplementary Notes
    RVV Implementation: Optimizing countWithinDistance for RANSAC Models
    RVV Implementation: 3D Vector Acute Angle Calculation (getAcuteAngle3D & acos)

  2. Migration of PR #3519 (SSE/AVX code): Implemented countWithinDistanceRVV for the Plane, NormalPlane, Circle2D, and Sphere models.

  3. Testing:
    Test Documentation
    Test Implementation

Can I submit a PR? Do I need to create a new branch in my repository and only submit the added code parts?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing PR #3519 and the feature/rvv10 branch, especially the sample_consensus implementation and test-rvv/sample_consensus tests. Read the linked RVV documentation and confirm the expected contribution process with maintainers; done means an accepted RVV implementation and corresponding tests that meet PCL requirements.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-vision, performance
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.