PointCloudLibrary/pcl

Replace pcl::visualization::createPlane and pcl::visualization::PCLVisualizer::addPlane overloads by default arguments

Ouverte

#1 653 ouverte le 18 juil. 2016

Voir sur GitHubGuide contributeur
 (6 commentaires) (0 réaction) (0 personne assignée)C++ (4 506 forks)batch import
good 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

The code is pretty similar and can be probably reduced by simply making a change to its signature with default initializers.

bool addPlane (const pcl::ModelCoefficients &coefficients, 
               double x = 0,
               double y = 0,
               double z = 0,
               const std::string &id = "plane",
               int viewport = 0);

and

PCL_EXPORTS vtkSmartPointer<vtkDataSet> 
createPlane (const pcl::ModelCoefficients &coefficients, double x = 0, double y = 0, double z = 0);

pcl::visualization::PCLVisualizer::addPlane method:

pcl::visualization::createPlane method:

Guide contributeur

Direction de recherche
Analysez les deux surcharges existantes de addPlane et createPlane, puis combinez les en une seule fonction avec des arguments par défaut. Mettez à jour tous les appelants pour utiliser la nouvelle signature.
Stack technique
cpp
Domaine
computer 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.
Ancienne
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.
65