Get rid of cstagger, use python module with numba (re-add #31)

ITA-Solar / helita

A Python library for solar physics from the Institute of Theoretical Astrophysics, University of Oslo

BSD 3-Clause "New" or "Revised" License

9 stars 17 forks source link

Get rid of cstagger, use python module with numba (re-add #31) #33

Closed tiagopereira closed 1 year ago

tiagopereira commented 3 years ago

This will get rid of all cstagger machinery, cython compilation, etc. and enable simpler functions for the stagger up/down operations.

DO NOT MERGE. Still needs some cleaning up.

M1kol4j commented 3 years ago

'stagger.py' works great but default padding in z-direction is reflect, while linear extrapolation would be a better choice. Unfortunately np.pad() does not support extrapolation by default but allows to define padding function. I think it would be crucial to define one with linear (log linear too?) extrapolation.

tiagopereira commented 1 year ago

This was addressed in #43 .