Present at review:
Question / concerns / comments
Enter your thoughts on the API and any questions / concerns you have here. Please sign your name. Anything you want to address in the API review should be marked down here before the start of the meeting.
- Configure pub rate?
Add parameter in aggregator constructor to allow pub rate to be configured (X)
Make Analyzer a pure virtual class (X)
Put list/string extraction stuff in GenericAnalyzer init into one place (X)
bool getParamVals(string param_name, vector<string> &output)
Use DiagnosticStatusWrapper if necessary. Not needed now, but could be useful for others. Avoids users doing stringstream (Not yet, we'll see)
Something about optimizing (Not yet, not API)
To be filled out by proposer based on comments gathered during API review period
Package status change mark change manifest)
Action items that need to be taken.
Major issues that need to be resolved