kinverarity1 / lasio

Python library for reading and writing well data using Log ASCII Standard (LAS) files
https://lasio.readthedocs.io/en/latest/
MIT License
342 stars 152 forks source link

Comparison over 100 wells between version 0.3 and 0.5.2 #42

Closed VelizarVESSELINOV closed 9 years ago

VelizarVESSELINOV commented 9 years ago

version 0.5.2 is able to load "XXX" mnemonic when is defined in the LAS file as ".XXX" :+1:

~Curve Information Block
DEPT.FT            0 000 00 00: Depth
GR.GAPI                       : Gamma Ray
TT3.USEC                      : CBL 3' Travel Time
AMP3.MV                       : CBL 3' Amplitude
AMP5.MV                       : CBL 5' Amplitude
ATT3.DB/FT                    : CBL 3' Attenuation
BONDIX.                       : CBL Bond Index
.CCL.                         : Casing Collar Locator
~Parameter Information Block
~A  Depth        GR         TT3         AMP3        AMP5        ATT3       BONDIX       .CCL    
  4468.0000   -999.2500   -999.2500   -999.2500   -999.2500   -999.2500   -999.2500   -999.2500

but version 0.5.2 is not able load several occurrences of "XXX" :-1: I prefer the old way, and you?

~Curve Information Block
#MNEM.UNIT               :   Curve Description
#---------                   ------------------------------
 DEPTH.F                 :     1  MEASURED DEPTH
 CAL.INCH               :     2  X CALIPER
 CAL.INCH               :     3  Y CALIPER
~Parameter Information Block
#MNEM.UNIT       Value   :   Description
#---------      ----------   ------------------------------
 CS  .INCH          5.500:   CASING SIZE
 CW  .LB/F          20.00:   CASING WEIGHT
 CG  .               N-80:   CASING GRADE
~A   DEPTH      CAL       CAL 
  9159.2998    -999.25    -999.25
  9158.7998    -999.25    -999.25
  9158.2998    -999.25    -999.25

in version 0.3 the channels DEPT, ROP and TGAS was correctly loaded, but not any more parsed with version 0.5: :-1: I prefer the old way, and you? I would even suggest to have default name, like "UNKNOWN" and when the mnemonic is empty you can replace it with "UNKNOWN" so you can load the last 3 channels with UNKNOWN[0-2].

~CURVE INFORMATION
#MNEM.UNIT           APICODE             CURVE DESCRIPTION
#---------         -----------           ------------------
DEPT.FT                     : 1 DEPTH
ROP.MIN/FT                  : 2 ROP
TGAS.UNITS                  : 3 TGAS
.                           : 4 WOB
.                           : 5 RPM
.                           : 6 BHF
~A DEPTH      ROP           TGAS                                                     
 7440          0             0             0             0             0             
 7442          0             0             0             0             0             
 7444          0             0             0             0             0             
kinverarity1 commented 9 years ago

I agree yes in principle. I will write a specific test for this to ensure I don't go backwards in support as you've found. Sorry.