# PhotoCalib produces negative flux errors for negative flux measurements.

XMLWordPrintable

#### Details

• Type: Bug
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
• Story Points:
1
• Sprint:
AP S19-1
• Team:

#### Description

The PhotoCalib, instFluxToMaggies method is missing an absolute value causing calibrated flux errors to be negative when fluxes are negative (as can happen in a difference image).

#### Activity

Hide
John Parejko added a comment -

Also, as part of this I should refactor the flux error calculation to take out the division by instFlux, so we don't get NaN flux for 0 counts.

Show
John Parejko added a comment - Also, as part of this I should refactor the flux error calculation to take out the division by instFlux , so we don't get NaN flux for 0 counts.
Hide
John Parejko added a comment -

I'm going to do this as part of DM-16650, which is going on now.

Show
John Parejko added a comment - I'm going to do this as part of DM-16650 , which is going on now.
Hide
John Swinbank added a comment -

“Won't Fix“-ing as a duplicate of DM-16650, which isn't quite true (this is a subset, rather than a duplicate), but it'll do.

Show
John Swinbank added a comment - “Won't Fix“-ing as a duplicate of DM-16650 , which isn't quite true (this is a subset, rather than a duplicate), but it'll do.
Hide
John Parejko added a comment -

This is definitely not "won't fix": it's been done as part of DM-16650 (not a duplicate, it was just logical to do the work there), and will be marked "done" when that ticket is merged.

Show
John Parejko added a comment - This is definitely not "won't fix": it's been done as part of DM-16650 (not a duplicate, it was just logical to do the work there), and will be marked "done" when that ticket is merged.
Hide
John Parejko added a comment -

DM-16650 was merged, and thus this fix is now in place (with an updated unittest to check for it).

Show
John Parejko added a comment - DM-16650 was merged, and thus this fix is now in place (with an updated unittest to check for it).

#### People

Assignee:
John Parejko
Reporter:
Chris Morrison [X] (Inactive)
Watchers:
Chris Morrison [X] (Inactive), Eric Bellm, Jim Bosch, John Parejko, John Swinbank