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

Mc dense #270

Open
wants to merge 5 commits into
base: devoto_dev
Choose a base branch
from
Open

Mc dense #270

wants to merge 5 commits into from

Conversation

Elisa-Visentin
Copy link
Collaborator

Add the possibility to use MC line with high density for training for some specific sources (till now I am sure only about Boomerang, whose results have already been published; if other sources still not public must be added, maybe we can retrieve their names from a private file)

Copy link

codecov bot commented Nov 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.23%. Comparing base (e64a44c) to head (1aebe2a).

Additional details and impacted files
@@             Coverage Diff             @@
##           devoto_dev     #270   +/-   ##
===========================================
  Coverage       77.23%   77.23%           
===========================================
  Files              21       21           
  Lines            2614     2614           
===========================================
  Hits             2019     2019           
  Misses            595      595           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@jsitarek
Copy link
Collaborator

I think for Boomerang they are either way going to do something special with a mixture of standard and dense line

@Elisa-Visentin
Copy link
Collaborator Author

I contacted FC: trey do the training with dense, then, for diffuse analysis, they use the non-dense for full-enclosure IRFs, so they could (try to) at least use our DL2s if we produce them with dense (as for Dl3, they will have only the point-like ones from us, but they can produce their own with their IRFs)

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

Successfully merging this pull request may close these issues.

2 participants