search

JustinGOSSES / wellio.js

JavaScript for converting well-log standard .las file format to json format
https://justingosses.github.io/wellio.js/
MIT License
34 stars 5 forks source link

Fails when using file containing an ~OTHER section #9

Closed kinverarity1 closed 6 years ago

kinverarity1 commented 6 years ago

Currently fails to load a LAS file with an ~O section e.g. https://raw.githubusercontent.com/kinverarity1/lasio/master/tests/examples/2.0/sample_2.0.las

JustinGOSSES commented 6 years ago

Fixed the las2json() function in wellio.js file such that the ~O section is caught now and the test las file given above works. Commit here(https://github.com/JustinGOSSES/wellio.js/commit/00495c2e5a57a2600cfcdf0ed3b612e680c3d733).

However, I haven't run many test wells through wellio.js or created a test suite, which should be part of making the parsing more robot as noted in issue (https://github.com/JustinGOSSES/wellio.js/issues/5).