FIT: add time-calibration command line option#7416
Conversation
AllaMaevskaya
left a comment
There was a problem hiding this comment.
Good work, but I can not imagine case when we need to calibrate data outside 2.5 ns (200 channels) range. It could be special artificial laser runs. I prefer to keep narrow range to remove tails and signals from slow particles
|
Hi @shahor02 |
|
@HimanshuCERN those which failed are unrelated, let's wait for others to converge. |
|
Hi @shahor02 |
AllaMaevskaya
left a comment
There was a problem hiding this comment.
Could you change
int FT0ChannelTimeTimeSlotContainer::sGausFitBins = 999;
to 100?
Results of fitting in so wide interval are wrong, I checked
|
Hi |
|
Error while checking build/O2/fullCI for e81aebf at 2021-11-03 00:12: Full log here. |
|
@jotwinow |
|
I merged this branch with recent dev and it passed all tests |
FIT: add time-calibration command line option
No description provided.