Mariyan requested a merge.
Updated by Rogers, Chris over 11 years ago
- File KLChannelMap-coverage.png KLChannelMap-coverage.png added
- File KL-data-structure-coverage.png KL-data-structure-coverage.png added
- File MapcppKLCellHits-coverage.png MapcppKLCellHits-coverage.png added
- File MapCppKLDigits-coverage.png MapCppKLDigits-coverage.png added
- Assignee changed from Rogers, Chris to Bogomilov, Mariyan
Test coverage is in general quite poor. I attached some screenshots of the unit test coverage for your code (generated using lcov). It looks like you never wrote any unit tests, perhaps there is some problem and the test script didn't find them? You need to please fix this before I can do the merge, or give me a ring if you have any questions. More information on unit tests is available at Unit_tests. Also I would be grateful if you could supply a page of latex on the KL reconstruction routines in, say,