PointCloudLibrary/pcl

Access to `points` should be suppressed in PCL code

Offen

#4.213 geöffnet am 23.06.2020

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (4.506 Forks)batch import
effort: mediumgood first issuekind: todopriority: gsoc

Repository-Metriken

Stars
 (9.023 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 8T 7h) (3 gemergte PRs in 30 T)

Beschreibung

Access to points can also be suppressed in a lot of places.

Originally posted by @SergioRAgostinho in https://github.com/PointCloudLibrary/pcl/pull/4190

My observations:

points usage can be removed in following ways:

  • replace with functions like: copyPointCloud instead of copyPoint
  • replace with functions like isNormalFinite
  • replace points. to use functions of PointCloud

#4217, #4218, #4219, #4221 tackle third type to a large extent

Contributor Guide

Research-Richtung
Identifizieren Sie Stellen im PCL Code, an denen direkter Zugriff auf das 'points' Mitglied verwendet wird, und ersetzen Sie sie durch äquivalente Funktionen wie copyPointCloud, isNormalFinite oder PointCloud Memberfunktionen.
Tech Stack
cpp
Domain
computer graphics
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70