-
Notifications
You must be signed in to change notification settings - Fork 27
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
Add 3D effective area to the specs #73
Comments
@TarekHC - I think it's a good addition. In principle it's a straightforward addition, we have the same situation for background: To add it to the spec, the main complication is probably that the AEFF page is already more complicated, and you'll have to find a way to organise the content so that it's clear, but not too much duplication for I haven't thought much about how to do it. Feel free to make a pull request directly, or if you want feedback before writing it, put a proposal or concrete questions here. |
What about the science tools? It shouldn't be too problematic either, right?
Ok good, I'll do that in ~3 weeks. |
My understanding is that this is the state for both Gammapy and Gammalib:
See http://gamma-astro-data-formats.readthedocs.io/en/latest/general/coordinates.html#field-of-view I'm not aware of a better suggestion to get more precise It will take some time to get it in the spec, exporters, CTA pipeline and science tools. Maybe @jknodlseder or @registerrier or @bkhelifi or @kosack or @ParsonsRD have an opinion here or even a better idea how to store more precise AEFF at the DL3 level for HESS, MAGIC, CTA, ...? |
DETX and DETY are in reconstructed coordinates, while effective area should be a function of true incident angle. Are you proposing to have a coordinate system for true coordinates that is somehow aligned with the array coordinates?
|
Well, I guess it must be aligned with the stronger performance dependencies. And in IACTs, the zenith/azimuth (specially zenith within CTA's FoV) affect the AEFF very strongly. We need to think of a way to add this dependency. This |
You may also be able to get away with far less bins if you choose a polar coordinate system for this instead of DETX/DETY, like (R, phi), since to first approximation it's radially symmetric, and then there is a phi-dependence that would generally be second-order and only very prominent with 2 telescopes or a strange layout during construction. Therefore having all Aeff lookups be a function of r with a phi dependence that could be simply a single bin (radially symmetric) or many bins if not. That way you don't even need two types of lookup, just one but where the number of bins is variable. |
This may be true for H.E.S.S. but is it still true for CTA with a larger FoV or if you use a divergent pointing mode that may be 10-20° wide?
We should make sure that the format covers all cases, and allows the low systematic uncertainties imposed for example by CTA requirements.
… Le 14 nov. 2016 à 11:12, Karl Kosack ***@***.***> a écrit :
You may also be able to get away with far less bins if you choose a polar coordinate system for this instead of DETX/DETY, like (R, phi), since to first approximation it's radially symmetric, and then there is a phi-dependence that would generally be second-order and only very prominent with 2 telescopes or a strange layout during construction. Therefore having all Aeff lookups be a function of r with a phi dependence that could be simply a single bin (radially symmetric) or many bins if not.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#73 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AC2oV96V8vbGyoq-rMWFgmIkzpX7Flykks5q-DQIgaJpZM4Ku3rA>.
|
I believe that what @kosack suggests makes sense. Using square bins in DETX and DETY can be approximately correct, but similar bins in (R,phi) would probably make more sense, and would simplify interpolations (even for a non-symmetric layout of telescopes, or divergent pointing). That would allow bigger bins for similar systematic uncertainty level, which would reduce the required amount of MC statistics, which is a sensitive matter for CTA... We currently support 3D background using DETX and DETY, and I think also in this case it makes more sense to use THETA and PHI. So now the question is: do we need a different format for a 3D background using THETA and PHI as FoV coordinates (e. g. |
Yes. That's how it works at the moment.
That's how it could work in the future. Science tools could be aware of certain axes (see current list here) and know how to transform those into their internal dataspaces used in analysis (mostly LON, LAT, ENERGY) and possibly have clever axis-specific methods for interpolation (e.g. circular for PHI here). @TarekHC - You've thought about this a lot and prototyped in flexIRF. I think any contributions in that direction to the spec or existing code (ctapipe or science tools) would be welcome. The way I see it is that what we wrote down now is because "something is better than nothing" and that's what we were using already, not because it's a great, flexible system that is the final solution. |
That may be the best solution for now.
The problem is that, up to now, I always had the impression that we did not want a flexible data format. What I understood is that the format should be simple, stable and "solid", while the software could be more flexible (but not the specs). Also, regarding flexIRF, I mostly focused on storing those IRFs, and not "reading" them (i. e. interpolating), which is probably the trickiest part. I am already starting to play with CTA IRF generation within ctapipe, but still everything is extremely preliminary. But yes, I should get more involved also in the science tools side, as it is not clear at all who does what (e. g. background calculation). |
On a side note, I think we should call them not by their shape but find more appropriate names. E.g. 2d could be |
Just to add: we probably then can use the same format also for background and psf tables that are not radially symmetric in the FOV |
Coordinate systems are independent of whether we express an estimated or a true quantity in them. |
Hi all,
I have two main motivations to add a
aeff_3d
to the specs. and also support within science tools (3D:ENERGY
,DETX
andDETY
):It shouldn't be a lot of effort to include this neither to the specs nor the science tools. What do you think, would it make sense? I could work on it 2/3 weeks from now.
Cheers!
The text was updated successfully, but these errors were encountered: