Joseph (Joe) Osborn
Joseph (Joe) Osborn
@asalzburger So that this is documented with this issue, here's a summary of my test and current problem so far. To create the XYZ field, I take our 2D rz...
If I change the relevant section in `BFieldMapUtils.cpp` to the following: ``` auto binNo = localToGlobalBin({{i-1, j-1, k-1}}, nIndices); if(binNo >= bField.size()) { binNo = bField.size() - 1; } grid.atLocalBins(indices)...
Looking at the code it isn't clear to me why this causes a seg fault in the low occupancy QA. I would suggest just checking out the branch yourself locally...
This change really hurts the efficiency at low pT, presumably because we are rejecting real tracks in addition to all of the ghost tracks that it removes. Have you studied...
This kills the efficiency at low pT again - did that fix get removed inadvertently?
This is probably a result of the cvmfs issues. Hold off pushing anything
Superseded by https://github.com/sPHENIX-Collaboration/coresoftware/pull/3090
Okay I will hold off on merging this then. It may be good to double check with Evgeny, as he was seeing a lot of unusual changes in the simulation...
> Do you know where the segfault happens? GDB says it happens when calling the geometry identifier in this line. I checked this myself also by adding in print statements...
> Would be great if you could check it against the current version. Potentially you can just update this file so you don't have to deal with breaking changes. I'll...