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

Some modification required for source-dependent MC production #191

Open
chaimain opened this issue Mar 6, 2022 · 9 comments
Open

Some modification required for source-dependent MC production #191

chaimain opened this issue Mar 6, 2022 · 9 comments

Comments

@chaimain
Copy link
Contributor

chaimain commented Mar 6, 2022

Following the discussion in #188 we will need to add --source-dep option for generating IRFs, and maybe some more things, as the discussion with @SeiyaNozaki progresses, regarding producing source dependent MC production.

@LukasNickel
Copy link
Member

Choosing pointlike gammas for RF right?
Anything else?
Lets discuss it in the call

@SeiyaNozaki
Copy link
Contributor

I think we need to modify for source-dependent analysis:
https://github.com/cta-observatory/lstmcpipe/blob/master/lstmcpipe/stages/mc_train.py#L52-L53
to use point gamma MC as RF training input
and
https://github.com/cta-observatory/lstmcpipe/blob/master/lstmcpipe/stages/mc_dl2_to_irfs.py#L284
to add --source-dep flag if soure-dep analysis is selected

@LukasNickel
Copy link
Member

Pointlike gammas for all random forests. Is that a good idea?

Is it enough to put source dependent in the config and use the normal for dl1/dl2 or is there something missing?

@SeiyaNozaki
Copy link
Contributor

Pointlike gammas for all random forests. Is that a good idea?

yes, this is the way for source-dependent analysis. It would make some bias for proton/electrom MC, but we can obtain better performance for point gamma source.

Is it enough to put source dependent in the config and use the normal for dl1/dl2 or is there something missing?

It should be enough only to activate source-dep flag in the config.

@vuillaut
Copy link
Member

I believe this can now be easily supported by the new PathConfig logic.
Is there still a need @SeiyaNozaki @chaimain ?

@vuillaut
Copy link
Member

vuillaut commented Dec 5, 2022

Hi @SeiyaNozaki
With the AllSky production, what gamma do you use for training for the source-dep analysis?
(since the TrainingDataset includes only GammaDiffuse)

@SeiyaNozaki
Copy link
Contributor

For all sky MC, I'm using diffuse gamma as RF training for source-dependent analysis (though I used point gamma before all sky MC).

@vuillaut
Copy link
Member

vuillaut commented Dec 5, 2022

For all sky MC, I'm using diffuse gamma as RF training for source-dependent analysis (though I used point gamma before all sky MC).

ha ok great, so #375 is in line with your processing and it also means there is no changes required in lstmcpipe to process source-dependent !!
You can request a prod for your source-dep analysis, the only difference is in the lstchain config.

@SeiyaNozaki
Copy link
Contributor

Hi @vuillaut ,
I want to ask base production for source-dependent analysis (like #449 ), but in general, I also want to improve the code for source-dependent analysis:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants