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.

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

ignore_data=True no longer works properly #580

Open
rct01 opened this issue Oct 2, 2023 · 1 comment
Open

ignore_data=True no longer works properly #580

rct01 opened this issue Oct 2, 2023 · 1 comment
Labels

Comments

@rct01
Copy link

rct01 commented Oct 2, 2023

In version 0.27 using ignore_data=True in lasio.read would result in the file headers being read incredibly quickly. Some timings:
[2023-10-02 11:37:12] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1.las : 36
[2023-10-02 11:37:12] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_2.las : 6
[2023-10-02 11:37:12] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_3.las : 4
[2023-10-02 11:37:12] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_4.las : 4
[2023-10-02 11:37:12] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_5.las : 6
[2023-10-02 11:37:12] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_6.las : 6

Since version 0.28 ignore_data=True has far less effect. Opening the same files in 0.28+
[2023-10-02 11:34:59] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1.las : 172
[2023-10-02 11:34:59] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_2.las : 105
[2023-10-02 11:34:59] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_3.las : 74
[2023-10-02 11:34:59] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_4.las : 133
[2023-10-02 11:34:59] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_5.las : 57
[2023-10-02 11:34:59] [INFO ] Time to open file - Collector 03_12_2020 20_55_04_DOMINIC 1_6.las : 87

My application is designed to scan hundreds of files and determine their contents. This issue means I cant upgrade past 0.27. Any help would be much appreciated.

@rct01 rct01 added the bug label Oct 2, 2023
@kinverarity1
Copy link
Owner

kinverarity1 commented Oct 2, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants