-
Notifications
You must be signed in to change notification settings - Fork 6
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
Issue508 performance tests #509
Conversation
…n easier format to compare runs
…functionality, without a parameter/with the default of false, it keeps clusters between packets and adds to them if the first readout(s) in next packet belong there, or starts new clusters if time gap is large enough.
…a full flush occurs, otherwise builder keeps clusters in case future packets contain relevant readouts.
… in this branch as an experimental approach to clustering over packets - not needed in the end
Hit hit; | ||
hit.coordinate = 50; | ||
hit.weight = 50; | ||
hit.time = 50; |
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.
indent
@@ -85,7 +85,7 @@ CSPECBase::CSPECBase(BaseSettings const &settings, | |||
Stats.create("readouts.tof_neg", Counters.TimeStats.TofNegative); | |||
Stats.create("readouts.prevtof_count", Counters.TimeStats.PrevTofCount); | |||
Stats.create("readouts.prevtof_neg", Counters.TimeStats.PrevTofNegative); | |||
Stats.create("readouts.tof_toolarge", Counters.TOFErrors); | |||
Stats.create("readouts.tof_toolarge", Counters.TOFErrors); //move this to events.tof_toolarge |
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.
Or discard at readouts stage
test/performancetest.py
Outdated
stats[stat_name] = metrics.return_metric(stat_name) | ||
return(stats) | ||
|
||
def efu_handles_data(efu, generator, module, throttle, packets_dropped): |
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.
needs renaming
|
||
from EFUMetrics import Metrics | ||
import argparse | ||
import csv |
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.
Move to EFUStat.py
Issue reference / description
The branch you merge from should already reference an event-formation-unit github ticket number. You can add a descriptive title, but if an issue is referenced, you don't have to.
Checklist for submitter
Nominate for Group Code Review (Anyone can nominate it)
Indicate if you think the code should be reviewed in a Thursday code review session.
Also, nominate it on the code_review Slack channel.