density_fit.py breaks backword compatibility

beam-tracing / Scotty

Beam tracing code for diagnostics

https://scotty.readthedocs.io/en/latest/

GNU General Public License v3.0

7 stars 4 forks source link

density_fit.py breaks backword compatibility #50

Closed valerian-chen closed 2 years ago

valerian-chen commented 2 years ago

When I try to run Scotty_main_sweep_MAST-U_45177.py, I get the following error

File "c:\users\vh chen\appdata\local\programs\python\python39\lib\site-packages\scotty\density_fit.py", line 374, in density_fit name, fit_method = FIT_LENGTH_MAPPINGS[len(parameters)]

TypeError: object of type 'WindowsPath' has no len()

density_fit should be choosing to load from ne.dat if no parameters are given.

ZedThree commented 2 years ago

I have a fix, PR incoming shortly, just trying to see if I can get a test on this