Allow for different thresholds for OpHitFinder

[asanchezcastillo]

SBND optical reconstruction requires using individual threshold for each channel. This PR incorporates the required changes which are summarized below:

• Setting an individual individual threshold for each channel requires carrying the channel number information until the specific reconstruction algorithm. Even thought RunHitFinder takes a raw:OpDetWaveform this is later converted to a pmtana::Waveform_t on the Reconstruct method of PulseRecoManager losing the required information about the channel number. The same happens for the Reconstruct method of PMTPulseRecoBase and the RecoPulse method implemented by the reconstruction algorithm. Fortunately enough, both pmtana::Waveform_t and raw::OpDetWaveform inherit from std::vector<> which to the best of my knowledge makes backwards compatibility given just by changing the object we pass into the former methods.
• Change the relevant peak finder algorithms to allow for individual threshold setting. The lists of threshold per channel must be passed through a fhicl parameter.

[FNALbuild]

A new Pull Request was created by @asanchezcastillo (Alejandro Sánchez Castillo) for develop.

It involves the following packages:

larana

@LArSoft/level-1-managers, @LArSoft/level-2-managers can you please review it and eventually sign? Thanks.

cms-bot commands are listed here

[FNALbuild]

The code-checks are being triggered in jenkins.

[FNALbuild]

-code-checks
Pull request failed code-formatting checks. Please ensure that cetmodules has been setup and execute the following command from the top-level directory of your repository:

format-code \
  larana/OpticalDetector/OpHitFinder/AlgoCFD.cxx \ 
  larana/OpticalDetector/OpHitFinder/AlgoSiPM.cxx \ 
  larana/OpticalDetector/OpHitFinder/AlgoSlidingWindow.cxx

Then commit the changes and push them to your PR branch.

[FNALbuild]

Pull request #36 was updated. @LArSoft/level-1-managers, @LArSoft/level-2-managers can you please check and sign again.

[FNALbuild]

The code-checks are being triggered in jenkins.

[FNALbuild]

+code-checks

[knoepfel]

@asanchezcastillo, do you intend this PR to be in draft state?

[asanchezcastillo]

@knoepfel I am marking it as ready for review now. Thanks for asking.