Skip to content

Increase LUT size to 200 and add spice kernel pipeline #32

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.

Sign up for GitHub

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

Merged
WillMatthews merged 1 commit into from
Jun 15, 2026
Merged

Increase LUT size to 200 and add spice kernel pipeline #32

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 9 additions & 3 deletions src/sipm.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,9 @@ SiPM::SiPM(unsigned long numMicrocell_in, double vbias_in, double vBr_in, double
microcellTimes = vector<double>{}; // microcell live time of last detection vector
microcellTimes.reserve(numMicrocell + 8); // Prevent issues with free()

LUTSize = 20; // Look Up Table Size
LUTSize = 200; // LUT size: 20 points biased the static transfer curve by up to
// ~3% at deep saturation (linear-interp error in eta/V); 200 keeps
// the bias below the MC noise floor (~0.1%) at negligible cost.
tVecLUT.assign(LUTSize, 0.0); // Preallocate LUT Time array
pdeVecLUT.assign(LUTSize, 0.0); // Preallocate LUT PDE array
vVecLUT.assign(LUTSize, 0.0); // Preallocate LUT Voltage array
Expand All @@ -86,7 +88,9 @@ SiPM::SiPM(unsigned long numMicrocell_in, double vbias_in, double vBr_in, double
microcellTimes = vector<double>{}; // microcell live time of last detection vector
microcellTimes.reserve(numMicrocell + 8); // Prevent issues with free()

LUTSize = 20; // Look Up Table Size
LUTSize = 200; // LUT size: 20 points biased the static transfer curve by up to
// ~3% at deep saturation (linear-interp error in eta/V); 200 keeps
// the bias below the MC noise floor (~0.1%) at negligible cost.
tVecLUT.assign(LUTSize, 0.0); // Preallocate LUT Time array
pdeVecLUT.assign(LUTSize, 0.0); // Preallocate LUT PDE array
vVecLUT.assign(LUTSize, 0.0); // Preallocate LUT Voltage array
Expand Down Expand Up @@ -146,7 +150,9 @@ SiPM::SiPM(vector<double> svars)
microcellTimes = vector<double>{}; // microcell live time of last detection vector
microcellTimes.reserve(numMicrocell + 8); // Prevent issues with free()

LUTSize = 20; // Look Up Table Size
LUTSize = 200; // LUT size: 20 points biased the static transfer curve by up to
// ~3% at deep saturation (linear-interp error in eta/V); 200 keeps
// the bias below the MC noise floor (~0.1%) at negligible cost.
tVecLUT.assign(LUTSize, 0.0); // Preallocate LUT Time array
pdeVecLUT.assign(LUTSize, 0.0); // Preallocate LUT PDE array
vVecLUT.assign(LUTSize, 0.0); // Preallocate LUT Voltage array
Expand Down
Loading