PointCloudLibrary/pcl

Use strongly typed 'enum' instead of 'int' for properties in PCLVisualizer code

Offen

#1.692 geöffnet am 24.08.2016

Auf GitHub ansehenContributor Guide
 (11 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (4.506 Forks)batch import
effort: lowgood first issuekind: todomodule: visualization

Repository-Metriken

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

Beschreibung

Right now functions like setPointCloudRenderingProperties use an int to represent the property to be modified. This means that there is no way of knowing which properties are accepted by the function. A solution is to use strongly typed enum instead of int for properties. The issue was brought up in #1668.

Contributor Guide

Research-Richtung
Identifizieren Sie Funktionen im PCLVisualizer, die int für Eigenschaftsparameter verwenden, und ersetzen Sie diese durch einen stark typisierten Enum. Beachten Sie Issue #1668 für verwandte Diskussionen und mögliche Designentscheidungen.
Tech Stack
cpp
Domain
apicomputer 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.
80