Results 45 comments of Nicholas Earl

Running list of possible operations to support in arithmetic. - [ ] Absolute value - [ ] Power of base 10 (inverse log base 10) - [ ] Power of...

@SaOgaz Could you rename this issue to something like "Fill out testing infrastructure for basic arithmetic", and then open a new issue (something like "Implement extended arithmetic tasks") that we...

I'm more inclined to leave it to the user. For us to provide this functionality, we would need to support both internally applying equivalencies during arithmetic operations, and allowing the...

@hcferguson There are [a list of checks](https://github.com/astropy/specutils/blob/master/specutils/spectra/spectrum1d.py#L145) that get performed on the respective wcs objects, and depending on if one fails, will log an error and return without performing the...

@alexji Do you mind if I rebase and push directly to your branch with some low-level fixes?

@eblur This case _should_ already be supported with the `extrapolation_treatment` argument in the resamplers. By default it is `nan_fill`, but you can use `zero_fill` instead. There's [an example](https://specutils.readthedocs.io/en/stable/manipulation.html#splicing-combining-multiple-spectra) in the...

Thanks @janerigby! I think we can certainly do something at the specutils level to help compatibility, but this might also be a higher-level astropy question of getting `Quantity` objects to...

It is a good idea! You can file the feature request [here](https://github.com/astropy/astropy/issues/new/choose).

@rosteen can you provide an example of what you're doing? I'm playing around with the analysis functions, but am not getting any `NaN` values returned.

The flux array is all `NaN`s. Looking at the returned value of `data` when trying to call `get_data_from_viewer` on a subset shows that the flux value is `NaN`s even there....