mreutegg / laszip4j

The LASzip library ported to Java
GNU Lesser General Public License v2.1
34 stars 15 forks source link

Introduce MultiMMappedDataInput and create multiple ByteBuffers to me… #104

Closed mreutegg closed 9 months ago

mreutegg commented 9 months ago

…mory map files bigger than 2 GB

codecov[bot] commented 9 months ago

Codecov Report

All modified lines are covered by tests :white_check_mark:

Comparison is base (927d406) 20.46% compared to head (600195d) 20.92%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #104 +/- ## ============================================ + Coverage 20.46% 20.92% +0.45% - Complexity 754 759 +5 ============================================ Files 170 170 Lines 12472 12498 +26 Branches 2748 2753 +5 ============================================ + Hits 2553 2615 +62 + Misses 9388 9354 -34 + Partials 531 529 -2 ``` | [Files](https://app.codecov.io/gh/mreutegg/laszip4j/pull/104?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [...hub/mreutegg/laszip4j/laszip/ByteStreamInFile.java](https://app.codecov.io/gh/mreutegg/laszip4j/pull/104?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-c3JjL21haW4vamF2YS9jb20vZ2l0aHViL21yZXV0ZWdnL2xhc3ppcDRqL2xhc3ppcC9CeXRlU3RyZWFtSW5GaWxlLmphdmE=) | `76.19% <100.00%> (+47.07%)` | :arrow_up: | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/mreutegg/laszip4j/pull/104/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

oers commented 9 months ago

[ERROR] LASReaderTest.readExtraBytesShort:464 expected:<[0.80, 1.12, 1.00, 1.28, 1.52]> but was:<[0,80, 1,12, 1,00, 1,28, 1,52]> During Build on WIndows

mreutegg commented 9 months ago

Thanks for reporting the failure. The test depends on the system locale.

mreutegg commented 9 months ago

Fixed with https://github.com/mreutegg/laszip4j/pull/105

sonarcloud[bot] commented 9 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

96.7% 96.7% Coverage
0.0% 0.0% Duplication