Skip to content

Commit 1f1e00f

Browse files
lynnt20henrylay97
andauthored
add documentation about hacky fix to SpaceCharge source code
Co-authored-by: Henry Lay <h.lay@lancaster.ac.uk>
1 parent a8996c0 commit 1f1e00f

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

sbndcode/SpaceCharge/SpaceChargeSBND.cxx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -215,13 +215,19 @@ geo::Vector_t spacecharge::SpaceChargeSBND::GetPosOffsets(geo::Point_t const& po
215215
//larsim requires negative sign in TPC 0
216216
int corr = 1;
217217

218+
// ========================================================
219+
// This hack is to account for a known issue with the
220+
// space charge implementation for the 2D simulation.
221+
// See https://cdcvs.fnal.gov/redmine/issues/28099
222+
// This should be removed once the appropriate upgrades
223+
// have been implemented.
224+
// ========================================================
218225
if(f_2D_drift_sim_hack == true)
219226
corr = -1;
220227

221-
if (xx < 0) {
228+
if (xx < 0) {
222229
corr = -1;
223230
}
224-
225231

226232
double offset_x=0., offset_y=0., offset_z=0.;
227233
offset_x = corr*SCEhistograms.at(0)->Interpolate(xx,yy,zz);

0 commit comments

Comments
 (0)