Merge csiro-internal and csiro-robotics mains#53
Open
tomniac wants to merge 41 commits into
Open
Conversation
…ely. This saves having to find what the output file is called and load it into a viewer
Feature/view all results
Switches readLas/writeLas and LasWriter from libLAS to the laszip_api.h C API, adding LAS 1.4 support (formats 6-10 and 64-bit point counts) and replacing the legacy DataFormatId bit-mask checks for time/colour. Updates the build to find LASzip 3.x and drops libLAS from the devcontainer.
- README: replace libLAS + LASzip 2.0.1 build steps with a single LASzip 3.4.4 instruction, note LAS 1.4 / COPC support. - docker/Dockerfile: drop the libLAS build stage, bump LASzip to 3.4.4 and build it Release. - .devcontainer/Dockerfile: build LASzip in Release (unoptimized builds make LAZ decompression 5-15x slower). - Remove the now-unused cmake/FindlibLAS.cmake module.
…epended on even if the repo on the host has a different name.
….devcontainer/commands/compile_install.sh into PATH in devcontainer. Added --security-opt=label=disable to devcontainer run args.
Fix compiler warnings.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
After merging this with one of csiro-internal or csiro-robotics, it can be force pushed to the other and then they will both return to identical commits.
Files changed (single file, ignore whitespace)
Changes
See