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

Gap in tracks at boundary of induction 1 plane #769

Open
jzettle opened this issue Oct 22, 2024 · 3 comments
Open

Gap in tracks at boundary of induction 1 plane #769

jzettle opened this issue Oct 22, 2024 · 3 comments
Assignees

Comments

@jzettle
Copy link
Contributor

jzettle commented Oct 22, 2024

No description provided.

@jzettle
Copy link
Contributor Author

jzettle commented Oct 22, 2024

Similarly to issue #768, account for the induction1 plane gap that we do not simulation as a systematic. I am working on one way to do this with @jzennamo's support where we can filter the SimEnergyDeposits (SEDs) around the plane gap at z = 0 as a way to generate a detector variation sample for this first analysis. There is a LArSoft module in place to filter the SEDs and feed the filtered SED collection into the rest of the reconstruction using a fcl that can be converted into a variation fhicl. I have validated the SED collection is filtered correctly and we are working on validating the downstream pieces after the filtering module runs.

@jzettle
Copy link
Contributor Author

jzettle commented Nov 22, 2024

There are PRs addressing the filter module to use this for a systematic variation sample. Once they are merged and patch releases of sbncode v09_89_01_01 and icaruscode v09_89_01_01p03 are created we can create samples using this. It also includes the results of a study by @jzennamo (see https://sbn-docdb.fnal.gov/cgi-bin/sso/ShowDocument?docid=38701)

@jzettle
Copy link
Contributor Author

jzettle commented Dec 2, 2024

Created a new PR against a branched version of the sbncode tag release v09_89_01_01 as originally intended. This differs from the branch release/SBN2024A. We need to base from this release for the variation studies so we should proceed here anyway in order to use this for the first 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

1 participant