specutils icon indicating copy to clipboard operation
specutils copied to clipboard

Add unit check for sum vs integral in line flux + moments

Open rosteen opened this issue 1 year ago • 0 comments

Currently, line flux does an integral (well, a sum multiplying by dx, see https://github.com/astropy/specutils/issues/712 for more discussion), whereas the moment calculations are doing simple sums. I'm fixing the moment calculations to match line flux in https://github.com/astropy/specutils/pull/1141, but really for both of these we should have a check to do the simple sum in the appropriate case of flux units that are actually flux (e.g. W/m2) rather than the usual case of flux density (e.g. Jy) that requires the integral.

rosteen avatar May 28 '24 14:05 rosteen