14 colorisation des nuages de point étroits largeur ou hauteur 02m

IGNF / ign-pdal-tools

MIT License

5 stars 1 forks source link

14 colorisation des nuages de point étroits largeur ou hauteur 02m #15

Closed CharlesGaydon closed 1 year ago

CharlesGaydon commented 1 year ago

proposition de solution pour les deux soucis évoqués dans https://github.com/IGNF/ign-pdal-tools/issues/14

color: support colorization for <0.2m clouds (including height=0/width=0)
color: ceil width/height to have a bbox that contains all points

leavauchier commented 1 year ago

Ca me parait ok, mais je préfère attendre le retour de @gliegard là-dessus avant le merge, je pense qu'il a plus de recul que moi sur cette partie du code

CharlesGaydon commented 1 year ago

Merci !

(My bad pour le linting ; j'ai reconfiguré mon VS Code pour qu'il prenne la config du projet quand il y en aura une dans pyproject.toml, ce qui semble une bonne idée :))

leavauchier commented 1 year ago

J'ai rajouté une config black ici : https://github.com/IGNF/ign-pdal-tools/pull/18

CharlesGaydon commented 1 year ago

@leavauchier C'est rebasé sur dev, et j'ai appliqué black avec la bonne config sur color.py et test_color.py.

Au passage j'ai remarqué qu'il faudrait appliquer la config de black sur l'ensemble des tests :)