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

Need common drift velocity calculation in SBND and ICARUS #205

Open
brucehoward-physics opened this issue Nov 23, 2021 · 4 comments
Open
Assignees

Comments

@brucehoward-physics
Copy link
Contributor

With the merge of PR SBNSoftware/icarusalg#43 in icarusalg ICARUS switched to an alternate description of drift velocity (UseIcarusMicrobooneDriftModel: true). At the field/temperature currently set in the ICARUS parameters this is a fairly small change the velocity.

Filing an issue since the same change was not made in SBND, at least yet, so that we can remember to have any necessary discussion in the future and push the fhicl change there as well.

@wesketchum
Copy link
Contributor

Assigning to Mike and Filippo, though @marcodeltutto and others may also want to weigh in.

I've also targeted this for 2022B (spring release) since that's the critical juncture for joint SBND/ICARUS analysis pieces, but the sooner it can be decided/get in the better.

@wesketchum wesketchum changed the title Drift velocity calculation changed in ICARUS Need common drift velocity calculation in SBND and ICARUS Nov 24, 2021
@varanini
Copy link

I agree with @mrmooney that the ICARUS change is both good and should have a fairly small impact on reconstruction. And should be extended to SBND.
Not in the position to comment much on the technical issues and timescale for the SBND change.

@ibsafa
Copy link
Contributor

ibsafa commented May 29, 2024

@brucehoward-physics has this issue been resolved?

@brucehoward-physics
Copy link
Contributor Author

This is a blast from the past. I'd leave that to @mrmooney as I'm not sure if the similar change was ever made in SBND or if they had those discussions. I'll also tag @varanini and @linyan-w as the other ICARUS and SBND calibration conveners, though this GitHub Issue predates Linyan joining so definitely not expecting her to have the answer! Just wanted to include relevant folks who could discuss amongst themselves.

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

5 participants