Support #596

Code review - MapCppTrackerDigitization

Added by Santos, Edward almost 13 years ago. Updated over 12 years ago.

Target version:
Start date:
22 July 2011
Due date:
05 August 2011
% Done:


Estimated time:


For code in src/map/MapCppTrackerDigitization/.


Updated by Rogers, Chris almost 13 years ago

  • Due date set to 05 August 2011
  • Category set to Tracker
  • Assignee set to Carlisle, Timothy
  • Target version set to Future MAUS release

Assigned to Tim. Note the code review guidelines at:

Get the code with command like:

bzr branch lp:~e-santos10/maus/devel


Updated by Santos, Edward almost 13 years ago

  • Subject changed from Code review request to Code review - MapCppTrackerDigitization

Updated by Carlisle, Timothy almost 13 years ago

  • Assignee changed from Carlisle, Timothy to Santos, Edward

Looks pretty good to me!

1) cpplint flagged up ~100 very minor errors: white space, spacing etc, overlong lines which could easily be tidied up, depending how rigidly we endorse cpplint..!

2) ADC / TDC aren't explained, so a brief outline would be enlightening for new members to MICE. Also adding them to: would be good too.

3) Why group in sets of 7 fibers? 7 fibers = 1 channel right? A short justification of this would save the user a trip to the TRD - I had to double check as I'd forgotten!

4) L117 SP in the comment: "enxct entry"

5) L191 a comment just to explicitly state what you're doing here with the time resolution would be nice. Same for the SciFidcFactor, as it doesn't mean very much to me.

6) L195 Could you add another line explaining what this means? I'm not well versed in the detector stuff, so a little more information would be helpful.



Updated by Rogers, Chris almost 13 years ago

We stick to the letter of cpplint I'm afraid. It's a pain, but it's the only way to make it work. can make exceptions by appending to



Updated by Rogers, Chris over 12 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

I think this is closed now.

Also available in: Atom PDF