SBNSoftware / icaruscode

Main/top level repository for ICARUS specific code
11 stars 34 forks source link

SpaceCharge service configurations #236

Open PetrilloAtWork opened 3 years ago

PetrilloAtWork commented 3 years ago

So far ICARUS is getting away with claiming space charge is negligible and with a SpaceChargeService configuration that just disables the corrections (fcl/services/correctionservices_icarus.fcl).

Space charge seems to be very popular anyway. Enough so that a proliferation of boilerplate configuration is happening in our configuration files. It is time to centralise the possible configurations into a single configuration file (e.g. spacecharge_icarus.fcl) and provide there some precooked configuratins (icarus_nospacecharge, icarus_nominalspacecharge and whatever), and link one of them to the "standard" configuration.

I don't have the expertise on the subject to decide the content of these configurations and how many of them are needed. More expert people than me (esp. from the SBN world) would have to contribute.

wesketchum commented 2 years ago

@mrmooney @varanini assigning you to coordinate on this if that's ok, and make sure we have something consistent for SBND.

@marcodeltutto and Alessandro M., good to keep you looped in ...

varanini commented 2 years ago

@mrmooney is more of an SCE expert than me. But I would agree (maybe adding a double space charge configuration for testing could be a good idea). We should be careful about not correcting for SCE twice (there was a similar problem for SBND at the recent calib workshop)

marcodeltutto commented 2 years ago

Thanks @wesketchum. We can actually work on having the same (or similar) configurations in sbndcode in a spacecharge_sbnd.fcl.

jzennamo commented 2 years ago

Just to clarify @mrmooney and @varanini are we happy that the current Stage1 reconstruction does not apply SCE corrections to the tracks in ICARUS?

varanini commented 2 years ago

OK for me

varanini commented 2 years ago

see conversation on Slack