Uploaded image for project: '1D Redmine '
  1. 1D Redmine
  2. REDMINE1D-400

[RM-8638] [linemodel] wrong total EmissionLinesSNR computation (and total StrongEmissionLinesSNR)

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Open (View Workflow)
    • Priority: Normal
    • Resolution: Unresolved
    • Labels:
      None

      Description

      Created on 2024-02-05 14:38:01 by Didier Vibert. % Done: 0

      The total SNR of emission lines (output as "EmissionLinesSNR") is wrongly computed, since the integrated flux of all emission lines is computed by subtracting the continuum from the observed spectrum, whereas the absorption lines fitted model should also be subtracted.
      cf @CLineModelFitting::getCumulSNRStrongEL()@ and @CLineModelFitting::getSNROnRange(Int32Range idxRange)@

      Moreover, the summation should follow a numerical integration like trapezoidal (using for instance @CSpectrumModel::integrateFluxes_usingTrapez@).
      The simplest fix would be to use the method @CLineModelFitting::getFluxDirectIntegration@ with the corresponding list of elements

      (bug detected when looking at the IT change of #6459)

        Attachments

          Activity

            People

            • Assignee:
              r2j.migrate Redmine-Jira Migtation
              Reporter:
              r2j.migrate Redmine-Jira Migtation
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: