D3DEnergetic / FIDASIM

A Neutral Beam and Fast-ion Diagnostic Modeling Suite
http://d3denergetic.github.io/FIDASIM/
Other
29 stars 19 forks source link

Test failure (in 2.0.0-pre): numpy int32 vs int64 types #197

Closed zhubr closed 4 years ago

zhubr commented 4 years ago

Hello, I'm getting this (in run_tests.py):

ERROR: "mask" has the wrong type of <class 'numpy.int32'>. Expected [<class 'num py.int64'>] ERROR: Invalid plasma parameters. Exiting...

Probably it has to do with various versions of libraries and python modules, anyway I could not find a solution yet. Numpy is 1.19.0 here currently. Not sure which other versions are relevant. Thanks.

zhubr commented 4 years ago

Changing np.int64 to np.int32 in nrnznphi_int assignment in preprocessing.py made tests complete normally. Still not sure if such change is correct and why it was necessary here. Thanks.

lstagner commented 4 years ago

Thanks for reporting this. I changed the mask type and run_tests.py works for me.