Action #3439
Add GSkyModel::flux() method that returns correct flux for diffuse cube models
| Status: | Closed | Start date: | 11/07/2020 | |
|---|---|---|---|---|
| Priority: | High | Due date: | ||
| Assigned To: |  Knödlseder Jürgen | % Done: | 100% | |
| Category: | - | |||
| Target version: | 2.0.0 | |||
| Duration: |
Description
Currently there is no method that computes correctly the flux for a diffuse cube model. flux() and eflux methods should be added to GSkyModel that correctly compute the flux for models where the integral over the spatial model is not unity. Variants should be included that allow provision of a sky region:
double flux(const GEnergy& emin, const GEnergy& emax) const;
double flux(const GEnergy& emin, const GEnergy& emax, const GSkyRegion* region) const;
double eflux(const GEnergy& emin, const GEnergy& emax) const;
double eflux(const GEnergy& emin, const GEnergy& emax, const GSkyRegion* region) const;
Recurrence
No recurrence.
Related issues
History
#1
Updated by Knödlseder Jürgen over 3 years ago
- Related to Bug #3440: Make sure that the spatial model value or normalisation is always taken correctly into account added
#2
Updated by Knödlseder Jürgen over 3 years ago
- Status changed from New to In Progress
- Assigned To set to Knödlseder Jürgen
- % Done changed from 0 to 90
The methods were implemented. Note that the methods with the GSkyRegion argument (that is now the first of the three arguments) correctly takes into account the normalisation factor of the spatial models, while this is not the case for the other methods.
I created issue #3440 to solve this issue before release 2.0.0.
#3
Updated by Knödlseder Jürgen over 3 years ago
- Status changed from In Progress to Closed
- % Done changed from 90 to 100
Code merged into devel.
#4
Updated by Knödlseder Jürgen over 3 years ago
- Related to Feature #3074: Add methods to calculate flux and energy flux errors from model parameter errors added