Different *.cfg breaks the library about pycomtrade HOT 4 CLOSED

Hello @ldemattos !

thanks for your help. If I understood correctly your comtrade record is broken, right? I agree with you that pyContrade should not crash in this case, but return to the user an indication of that. In you solution is this your idea or are you trying to read de broken comtrade? If think that a broken cfg should not be readed.

Best regards,

from pycomtrade.

Hi @miguelmoreto !

My pleasure to help!

I don't know if it is broken or those are optional fields, since the record that I'm analyzing opens correctly on other oscillographic readers. I'm not used IEEE Comtrade standard details. Sorry.

The solution that I have proposed simply check if the missing fields exists, and if TRUE, reads it, or else skip it. It's a very simple solution that do not hinder by no means the reading of files that contains all fields. Nevertheless, it make the program more reliable.

I do agree that a broken cfg should not be opened, however this one is a keeper.

Best regards,

from pycomtrade.

Ok. Can you include some comments in the code about these modifications? After that I will merge your PR. Best regards.

from pycomtrade.

Done! PR merged!

from pycomtrade.