Skip to content
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

Update pdal to 2.6+ #27

Merged
merged 1 commit into from
Oct 31, 2023
Merged

Update pdal to 2.6+ #27

merged 1 commit into from
Oct 31, 2023

Conversation

leavauchier
Copy link
Collaborator

@leavauchier leavauchier commented Oct 30, 2023

A priori passer à pdal 2.6+ règle des problèmes lié aux classification flags, visibles en particulier quand on utilise replace_attribute_in_las.py pour remplacer des classifications à partir d'un fichier las 1.2 (qui n'a des classes jusqu'à 31) par des valeurs > 31, ce qu'on devrait pouvoir faire puisque le format de sortie standard est 1.4 dans cette partie de la lib.

Des adaptations ont été necessaires sur les tests car :

  • le décompte des points ne prend plus en compte les flags, donc on n'a plus de valeurs > 31 dans count_occurences pour des fichiers las <=1.3
  • les classflags sont maintenant gérés indépendemment, donc au lieu d'une dimension ClassFlags, on a 4 dimensions Keypoint, Overlap, Synthetic, Withheld

@leavauchier leavauchier requested a review from gliegard October 30, 2023 15:43
@leavauchier leavauchier merged commit 3fa6c30 into dev Oct 31, 2023
1 check passed
@leavauchier leavauchier deleted the fix-replace-classes branch October 31, 2023 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants