Integration of Strips SR1 code
There is currently a branch devel_SR1 (see !553 (closed)), which implements several scans/analyses for Strips testing at SR1. For instance this is the status to which #173 refers.
I don't think it's practical to merge as is, the main things that should be fixed are:
- Need to load calibration information from a previous test into the histogram clipboard for fit functions etc. (see StarJsonData and HistoFromDisk). This should instead be done by putting the calibration information into the FrontEnd configuration.
- In order to have a noise occupancy scan with a variable number of triggers, the analysis includes code to communicate directly with the StarCounterLoop to change the number of triggers to be sent. Instead the analysis should use the existing feedback mechanism to request more triggers, and find a way to communicate how many triggers were actually sent.
The plan is to split the above branch into smaller pieces to be merged in turn. For instance, the following (subject to change as things go ahead):
-
Strobe delay analysis !564 (merged) -
Put calibration parameters into the configuration !584 (merged) -
Initial throttle trigger loop, maybe rebased !366 -
Make trigger throttle loop more robust -
Implement stripped down noise analysis (without the trigger throttle loop code) (depends on !584 (merged)). Originally in !497 (closed). -
Port the n-point gain analysis (with updates to fill configuration in !584 (merged)), see !689 (merged) -
Port the trim analysis (original: !490 (closed), new: !667) -
Feedback configuration of Strobe Delay (not part of devel_SR1 as it's done externally), with correct chip mapping related to #169 (the input channel mapping check) -
Loop action StarTrimDacLoop, should be possible to use StdParameterLoop instead -
Use Fuse ID to set HCC communications ID !665 (merged)
Other things that are in the branch that may or not be merged (might not be exhaustive). Mostly the code has been presented in other merge requests:
-
Addition to ScurveFitter to allow extra analysis !452 (closed) and !453 (closed) -
Adding integral to histo (I think this is redundant) !513 -
Mutex in TxCore (basically pairing command mask and send fifo, maybe add a function writeFIFOToCommand(mask, buffer)
) !556 (closed) -
NetIO/TxCore writeFifo 8-bit. Not sure why bytes, rather than 16-bit words are required? !391 -
Report of burst timing for StarCounterLoop -
Histogram file name changes ('_' vs '-' as separator) !455 (closed) -
Addition of LoopStatus to OccupancyMaps !437 (closed) -
HistoFromDisk which I think is not needed if calibration information is added to configuration !432 (closed) -
Broadcast write in StdParameterLoop (and prescan) potentially invalidating register fields (!477 (closed)) -
Some handling of fuse IDs (needed to set up HCC communication ID). Currently m_sn is in StarChips.h but never set (!665 (merged) including rename to make it clear SN is different)
Edited by Elise Maria Le Boulicaut Ennis