Confusing code in file sm.py - Githubissues

SEMAFORInformatik / femagtools

Python API for FEMAG

BSD 2-Clause "Simplified" License

20 stars 15 forks source link

Confusing code in file sm.py #89

Closed sasrigais closed 2 days ago

sasrigais commented 3 weeks ago

Hi all!

This issue is regarding checking bertotti losses in file machine/sm.py. Mainly in the range of lines 791.. and 911... keys is supposed to be a list. Later theres a call to keys.update() which does not excist for lists... A keys.append() should do the trick :)

Anyways there are too many curly braces in this call to keys.update().

Please look at lines 791... and 911... of file machine/sm.py.

Cheers Thomas

dapu commented 2 days ago

thank you Thomas, I corrected the confusion.