Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ECPL spectra #60

Closed
3 tasks done
cdeil opened this issue Dec 19, 2016 · 2 comments
Closed
3 tasks done

ECPL spectra #60

cdeil opened this issue Dec 19, 2016 · 2 comments

Comments

@cdeil
Copy link
Contributor

cdeil commented Dec 19, 2016

I just entered an ECPL spectrum:
749958e#diff-501fd4323310af93fb916ffb0b91d117R22

The spectral model parametrisation and parameters are here:
http://www.aanda.org/articles/aa/full/2006/47/aa5546-06/aa5546-06.html

I think the norm might be processed incorrectly, since currently the catalog has:

spec_norm_1TeV: 2.0170070474945234e-11

but the differential flux at 1 TeV should be lower, the difference being the exp factor.

What we need is

  • Start by checking this one source
  • Check all ECPL spectra in gamma-cat
  • Define a better spec result input format, or at least clearly document how the norm is processed for ECPL.
@cdeil
Copy link
Contributor Author

cdeil commented Dec 19, 2016

Actually, @adonath pointed out that this source is correct.

In [4]: np.exp(-1/24.8) * 21.0
Out[4]: 20.170070698235794

To avoid confusion, we'll rename to spec_dnde_1TeV.

I still think we should check the other spectra and define the format more clearly.
Keeping this issue open as a reminder...

@cdeil
Copy link
Contributor Author

cdeil commented May 12, 2017

With #119 and #122 the formats are better now.
(we need better docs and validation tools, but the formats are as clear as I could make them)

We're doing review of many spectra by plotting them and comparing to spectral points.
Of course, a systematic review of all ECPL or PL2 spectra would be nice, but I don't think anyone has the time and it's not clear that it's needed.
So closing this issue now.

@cdeil cdeil closed this as completed May 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants