search

SEMAFORInformatik / femagtools

Python API for FEMAG
BSD 2-Clause "Simplified" License
19 stars 13 forks source link

Order-numbers > 1000 break bch.Reader.read() #56

Closed sasrigais closed 3 years ago

sasrigais commented 4 years ago

FEMAG does not save order-numbers > 1000 in the BCH-File (Fe-Hysteresis- and Eddy current Losses for example). This produces nan values during parsing. Finally casting the order to int in mehtod __read_hysteresis_eddy_current_losses() will throw an cannot convert float NaN to integerError I think, that's not directly a femagtools issue or should be handeld by femagtools. It should be fixed in FEMAG instead, by allowing order numbers > 1000.

ronaldtanner commented 4 years ago

Your comment is correct. We will fix this in FEMAG

ronaldtanner commented 3 years ago

should be fixed now in the recent femag-classic version. please drop a note if not