Feat/compute ni n#33
Conversation
| limit_N_points: 400 | ||
| limit_flight_agl: 800 | ||
|
|
||
| z0: 0.0 |
There was a problem hiding this comment.
si tu commences à avoir beaucoup de paramètres è ajouter dans cette section, tu peux faire une sous-section strata par exemple, ce sera peut-être plus facile à terme
| assert cos_theta == 1.0 | ||
|
|
||
|
|
||
| def test_pad_metrics_core_returns_ni_n_min_layer_with_default_strata_shape(): |
There was a problem hiding this comment.
à mon avis tu peux merger ce test avec test_pad_metrics_core_returns_none_when_too_few_points pour en faire un test qui vérifie que les sorties ont bien le format attendu
There was a problem hiding this comment.
oups, je t'ai dit des conneries, je pensais merger avec test_pad_metrics_core_scanning_angle_false_returns_one seulement, histoire de ne lancer qu'une fois pad_metrics_core pour vérifier tous les formats de sortie à la fois, là ça fait moins de sens de merger aussi avec test_pad_metrics_core_returns_none_when_too_few_points vu qu'il faut lancer pad_metrics_core avec 2 jeux de paramètres différents, désolée
2 participants
Add two functions :
https://github.com/orgs/IGNF/projects/32/views/17?pane=issue&itemId=181841417&issue=IGNF%7ClidarHD_process%7C459 : step 3