-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Z on Virtual points #8
Conversation
467bb38
to
eb89910
Compare
|
||
Parameters: | ||
- points (numpy.ndarray): An array of 3D points [x, y, z] | ||
- q (int): Percentage or sequence of percentages for the percentiles to compute. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comme tu ne testes qu'avec une seule valeur de q dans l'appel de fonction, je pense que tu peux changer ton commentaire pour ne plus parler de séquence de pourcentages (sinon ça implique d'adapter le reste de ton code pour que ça marche aussi) :
- q (int): Percentage or sequence of percentages for the percentiles to compute. | |
- q (int): Percentage for the percentiles to compute. |
test/pointcloud/test_crop_las.py
Outdated
@@ -19,7 +19,7 @@ def setup_module(module): | |||
|
|||
|
|||
def test_crop_pointcloud_default(): | |||
classes = "[1:2]" | |||
classes = [2] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
normalement l'équivalent de ce que tu avais avant serait classes = [1, 2]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
c'est vrai, j'ai pas été rigoureuse !
Ca y est ! J'ai tout modifier ! On va pouvoir merger (jespère et donc créer une nouvelle branche pour les points virtuels)" |
Création de la fonction qui calcule un Z le long du squelette hydro
/ ! \ 1er étape, car il faut une deuxième étape de lissage du Z