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.

Sign up for GitHub

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

Jump to bottom

FEAT-865: LAZ writer #104

Merged
merged 55 commits into from
Apr 14, 2022
Merged

FEAT-865: LAZ writer #104

merged 55 commits into from
Apr 14, 2022

Conversation

leo-stan
Copy link
Contributor

No description provided.

@leo-stan leo-stan marked this pull request as draft March 31, 2022 22:28
@leo-stan leo-stan marked this pull request as ready for review April 5, 2022 04:31
clee-ai
clee-ai reviewed Apr 5, 2022
View reviewed changes
Copy link
Contributor

@clee-ai clee-ai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome stuff! (pt1)

cpp/include/copc-lib/io/laz_base_writer.hpp Outdated Show resolved Hide resolved
cpp/include/copc-lib/io/laz_writer.hpp Show resolved Hide resolved
cpp/include/copc-lib/io/laz_writer.hpp Show resolved Hide resolved
cpp/include/copc-lib/las/header.hpp Outdated Show resolved Hide resolved
cpp/include/copc-lib/las/laz_config.hpp Outdated Show resolved Hide resolved
cpp/include/copc-lib/las/laz_config.hpp Show resolved Hide resolved
cpp/src/io/copc_writer_internal.cpp Outdated Show resolved Hide resolved
cpp/src/io/copc_writer_internal.cpp Outdated Show resolved Hide resolved
clee-ai
clee-ai reviewed Apr 5, 2022
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
cpp/src/io/copc_writer_internal.cpp Outdated Show resolved Hide resolved
cpp/src/io/laz_base_writer.cpp Outdated Show resolved Hide resolved
cpp/src/io/laz_base_writer.cpp Outdated Show resolved Hide resolved
cpp/src/io/laz_writer.cpp Outdated Show resolved Hide resolved
cpp/src/io/laz_writer.cpp Outdated Show resolved Hide resolved
cpp/src/io/laz_writer.cpp Show resolved Hide resolved
clee-ai
clee-ai reviewed Apr 12, 2022
View reviewed changes
cpp/include/copc-lib/las/header.hpp Outdated Show resolved Hide resolved
cpp/include/copc-lib/las/header.hpp Outdated Show resolved Hide resolved
cpp/include/copc-lib/laz/compressor.hpp Show resolved Hide resolved
cpp/src/io/copc_writer_internal.cpp Show resolved Hide resolved
cpp/src/io/laz_writer.cpp Show resolved Hide resolved
cpp/src/las/header.cpp Outdated Show resolved Hide resolved
leo-stan added 22 commits April 12, 2022 16:15
@leo-stan
Rename CheckAndUpdateBounds and add SetGpsTimeBit.
18ec87b
@leo-stan
Refactor header space fill to BaseWriter and rename LasHeader::SIZE_B…
e843c4d 
…YTES to LasHeader::HEADER_SIZE_BYTES.
@leo-stan
Fix github test runs.
283fab8
@leo-stan
Merge branch 'test_fix' into laz_writer
1783a1c
@leo-stan
Update LasHeader GpsTime type bit tests.
187dbb1
@leo-stan
Merge branch 'test_fix' into misc_improvements
840ebc5
@leo-stan
Revert "Update LasHeader GpsTime type bit tests."
94fc920 
This reverts commit 187dbb1.
@leo-stan
Revert "Refactor header space fill to BaseWriter and rename LasHeader…
d404323 
…::SIZE_BYTES to LasHeader::HEADER_SIZE_BYTES."

This reverts commit e843c4d.
@leo-stan
Revert "Rename CheckAndUpdateBounds and add SetGpsTimeBit."
7f8309c 
This reverts commit 18ec87b.
@leo-stan
Revert "Revert "Refactor header space fill to BaseWriter and rename L…
f9b9fa3 
…asHeader::SIZE_BYTES to LasHeader::HEADER_SIZE_BYTES.""

This reverts commit d404323.
@leo-stan
Fix refactoring of std::fill_n.
90e5573
@leo-stan
Revert "Revert "Rename CheckAndUpdateBounds and add SetGpsTimeBit.""
30c9b92 
This reverts commit 7f8309c.
@leo-stan
Fix tests after CheckAndUpdateBounds name change.
a8c2df2
@leo-stan
Revert "Revert "Update LasHeader GpsTime type bit tests.""
40c7fde 
This reverts commit 94fc920.
@leo-stan
Merge branch 'laz_writer' into misc_improvements
c24b682
@leo-stan
Fix return type of LasHeader::ApplyInverseScale.
30f49e2
@leo-stan
Fix return type of LasHeader::ApplyInverseScale.
0c33ec0
@leo-stan
Fix bug in filepath in writer_test.py.
69266a3
@leo-stan
Remove remaining static_cast<long>.
c370bd6
@leo-stan
Revert fill_n changes.
12949f5
@leo-stan
Change type of LasHeader::RemoveScale from int64 to in32.
07555ad
@leo-stan
Merge branch 'laz_writer' into misc_improvements
15519f6 
# Conflicts:
#	cpp/include/copc-lib/io/laz_base_writer.hpp
clee-ai
clee-ai reviewed Apr 13, 2022
View reviewed changes
CHANGELOG.md Outdated Show resolved Hide resolved
example/example_reader.py Outdated Show resolved Hide resolved
example/example_reader.py Outdated Show resolved Hide resolved
example/example-reader.cpp Outdated Show resolved Hide resolved
example/example-reader.cpp Outdated Show resolved Hide resolved
cpp/src/copc/extents.cpp Outdated Show resolved Hide resolved
@clee-ai clee-ai merged commit a69ca4e into main Apr 14, 2022
@clee-ai clee-ai deleted the laz_writer branch April 14, 2022 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clee-ai clee-ai clee-ai approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@leo-stan @clee-ai