Skip to content

Proposal for changing the bitWiseScan in common tools

Mauro Dinardo requested to merge (removed):chipPolymorphism into chipContainer

I've implemented a slightly different version of the binary search in bitWiseScan

I think the main differences are the following:

(a) this version I'm sure is capable of exploring all possible values within the range of the register

(b) it starts from the pre-charged values in the register

This procedure works extremely well for me and it allows to set the final threshold to 900 electrons (starting from 2000 electrons) in only two steps

The previous procedure would first suggest moving to 400 Vth to 380 Vth, then in the second round would suggest moving back to 405 Vth

It is not intended for merging as it is. If you accept it, I think we should adapt the same for the global register. Moreover, right now I do not test the "direction", but I think that's can remain as it is and simply adapt this line accordingly: https://gitlab.cern.ch/cmsinnertracker/Ph2_ACF/blob/chipPolymorphism/tools/Tool.cc#L1031

Let me know.

  • Mauro.

Merge request reports