search

laslibs / lasrs

A rust library for parsing geophysical well log (.las) files
https://crates.io/crates/lasrs
6 stars 3 forks source link

Make the parser more robust against missing sections #10

Closed weiznich closed 2 months ago

weiznich commented 6 months ago

This commit removes all unwraps used in the parser code. Instead we just assume that the relevant sections are not there and return default values (empty lists) instead.

weiznich commented 3 months ago

@iykekings Would you be interested in reviewing and merging this PR?

iykekings commented 3 months ago

Yeah can you bump the version in Cargo.toml

weiznich commented 3 months ago

@iykekings Thanks for the fast response :heart: I pushed https://github.com/laslibs/lasrs/pull/10/commits/33d0a7321d82beb5923c8b329c44f30c5f2f8d8b to bump the patch version, given that this is a purely internal change

weiznich commented 2 months ago

@iykekings This is a gentle ping to get this moving again.