equinor / xtgeo

XTGeo Python class library for subsurface Surfaces, Cubes, Wells, Grids, Points, etc
https://xtgeo.readthedocs.io/en/latest/
GNU Lesser General Public License v3.0
110 stars 56 forks source link

ENH: improve i/o for IJXYZ format #1109

Closed jcrivenaes closed 8 months ago

jcrivenaes commented 8 months ago

Rewrite import and export from C (cxtgeo) to python, which allows for painless memory stream handling.

Keeps the C methods for a while as fallback. For export to file, the C code is twice as fast as the python code.

Closes #1081

codecov-commenter commented 8 months ago

Codecov Report

Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (35499de) 80.00% compared to head (c6a4319) 80.25%.

Files Patch % Lines
src/xtgeo/surface/_regsurf_export.py 80.00% 3 Missing and 2 partials :warning:
src/xtgeo/surface/_regsurf_import.py 66.66% 2 Missing and 1 partial :warning:
src/xtgeo/surface/regular_surface.py 40.00% 3 Missing :warning:
src/xtgeo/surface/_regsurf_ijxyz_parser.py 97.95% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1109 +/- ## ========================================== + Coverage 80.00% 80.25% +0.24% ========================================== Files 95 96 +1 Lines 13434 13514 +80 Branches 2185 2191 +6 ========================================== + Hits 10748 10845 +97 + Misses 1959 1940 -19 - Partials 727 729 +2 ```

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