Better `interpolate_along_los()`: plotting + `'l_from_impact'`

ToFuProject / tofu

Project for an open-source python library for synthetic diagnostics and tomography for Fusion devices

https://tofuproject.github.io/tofu/index.html

MIT License

69 stars 11 forks source link

Closed dvezinet closed 1 month ago

dvezinet commented 1 month ago

interpolate_along_los() now:
- accepts key_coords='l_from_impact'
- is more robust vs non-concatenated data
- allows to define all plotting features in dcolor (color, linestyle, linewidth, marker...)
- domain=dict is operational for 'l_from_impact' too

fixes, in devel, issue #936

pep8speaks commented 1 month ago

Hello @dvezinet! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 406:21: E303 too many blank lines (2) Line 811:5: E303 too many blank lines (2)