[DM-27810] Skip converting flux to amplitude in computeFixedMomentsFlux - Jira

XML

Word

Printable

Details

Type: Improvement
Status: Done
Resolution: Done
Fix Version/s: None
Component/s: meas_base
Labels:
None

Story Points:
1
Epic Link:
f20b-drp-gaap
Sprint:
DRP F20-5 (Oct)
Team:
Data Release Production

Description

It seems a bit silly to convert the flux weighted by the Gaussian to an amplitude, only to convert it back to weighted flux, when it can be done by passing a pointer directly to the `calcmom` function. It avoids having to divide and multiply by factors involving PI, although the loss in precision is negligible.

Attachments

Activity

Ascending order - Click to sort in descending order

No builds found.

Arun Kannawadi created issue - 01/Dec/20 9:23 PM

Arun Kannawadi made changes - 01/Dec/20 9:23 PM

Field	Original Value	New Value
Epic Link		~~DM-26794~~ [ 439744 ]

Arun Kannawadi made changes - 01/Dec/20 9:52 PM

Status

To Do [ 10001 ]

In Progress [ 3 ]

Hide

Permalink

Arun Kannawadi added a comment - 02/Dec/20 4:34 PM

Jenkins run: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/33178/pipeline/

Show

Arun Kannawadi added a comment - 02/Dec/20 4:34 PM Jenkins run: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/33178/pipeline/

Arun Kannawadi made changes - 03/Dec/20 5:49 PM

Reviewers		Jim Bosch [ jbosch ]
Status	In Progress [ 3 ]	In Review [ 10004 ]

Jim Bosch made changes - 03/Dec/20 5:58 PM

Status

In Review [ 10004 ]

Reviewed [ 10101 ]

Arun Kannawadi made changes - 03/Dec/20 6:50 PM

Resolution		Done [ 10000 ]
Status	Reviewed [ 10101 ]	Done [ 10002 ]

People

Assignee:: Arun Kannawadi

Reporter:: Arun Kannawadi

Reviewers:: Jim Bosch

Watchers:: Arun Kannawadi, Jim Bosch

Votes:: 0 Vote for this issue

Watchers:: 2 Start watching this issue

Dates

Created:: 01/Dec/20 9:23 PM

Updated:: 03/Dec/20 6:50 PM

Resolved:: 03/Dec/20 6:50 PM

Jenkins