-
Notifications
You must be signed in to change notification settings - Fork 59
Add EBLAPPD tool in EBV2 tool chain #336
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: Application
Are you sure you want to change the base?
Conversation
|
This looks good! New tool and no obvious problems. We have used this extensively and have not observed any problems. |
|
|
||
| bool EBLAPPD::Matching(int targetTrigger, int matchToTrack) | ||
| { | ||
| cout << "\033[1;34m******* EBLAPPD : Matching *******\033[0m" << endl; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
printout
| { | ||
| uint64_t LAPPDtime = MatchBuffer_LAPPDTimestamp_ns.at(i); | ||
| // if found LAPPDtime at PairedLAPPDTimeStamps, skip //shouldn't happen | ||
| if (std::find(PairedLAPPDTimeStamps[matchToTrack].begin(), PairedLAPPDTimeStamps[matchToTrack].end(), LAPPDtime) != PairedLAPPDTimeStamps[matchToTrack].end()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
count rather than find
| int matchedTrack = 0; | ||
| int matchedIndex = 0; | ||
|
|
||
| for (std::pair<int, std::vector<std::map<uint64_t, uint32_t>>> pair : GroupedTriggersInTotal) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use a reference to avoid copy
| // Log("EBLAPPD: Skipping TrackTriggerWord " + std::to_string(TrackTriggerWord), v_debug, verbosityEBLAPPD); | ||
| continue; | ||
| } | ||
| vector<std::map<uint64_t, uint32_t>> GroupedTriggers = pair.second; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use a reference to avoid copy
|
|
||
| for (int j = 0; j < GroupedTriggers.size(); j++) | ||
| { | ||
| map<uint64_t, uint32_t> groupedTrigger = GroupedTriggers.at(j); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use a reference to avoid copy
EBLAPPD Tool in EventBuildingV2 tool set.
This is also splitted from PR #307