rodluger / starry

Tools for mapping stars and planets.
https://starry.readthedocs.io
MIT License
142 stars 32 forks source link

`set_matrix` method is buggy #288

Closed rodluger closed 2 years ago

rodluger commented 3 years ago

Re-write this method; currently I'm seeing occasional memory leaks/garbage output, even though the unit tests are passing.

rodluger commented 2 years ago

I believe the issue has been fixed (I was indexing things using int8, which is like, a terrible idea). Users should feel free to re-open if they run into weird indexing issues (i.e., the map coefficients you specify are different from the ones you get!).