Thomas Robitaille

Results 891 comments of Thomas Robitaille

@Cadair could this be due to sunpy actually making something more correct so that we have to update the reference files?

This can be significantly simplified once https://github.com/astropy/extension-helpers/pull/26 is merged

Closing in favour of https://github.com/astropy/reproject/pull/506

@Cadair - I pushed the changes I suggested in case you agree with them

Ok so we need to add proper support for non_reprojected_dims to ``reproject_and_coadd`` rather than the current hack. And then of course a lot of documentation and tests!

Well this was useless as it passes and doesn't reproduce https://github.com/astropy/reproject/issues/491

Seems ok, probably fine to just merge without vote?

I wonder if the cython wrapping function could be more generally useful in other parts of astropy - for example for folding time series. Perhaps we should think of whether...

For fun: if we don't think we need the fused dtypes, we could also just use a plain C extension which compiles and runs faster: https://gist.github.com/astrofrog/136d15ebd650b4465060e314fccd7988 - runs about twice...

Could you provide the input and output headers you are using?