PointCloudLibrary/pcl

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

Ouverte

#1 692 ouverte le 24 août 2016

Voir sur GitHubGuide contributeur
 (11 commentaires) (0 réaction) (0 personne assignée)C++ (4 506 forks)batch import
effort: lowgood first issuekind: todomodule: visualization

Métriques du dépôt

Stars
 (9 023 étoiles)
Métriques de merge PR
 (Merge moyen 8j 7h) (3 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Identifiez les fonctions dans PCLVisualizer qui utilisent int pour les parametres de propriete et remplacez les par une enum fortement typee. Consultez le probleme #1668 pour les discussions associees et les choix de conception potentiels.
Stack technique
cpp
Domaine
apicomputer graphics
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80