perftool-incubator / CommonDataModel

Templates, scripts, and documentation on using a common data format for metrics, benchmark, and environment data
GNU General Public License v3.0
1 stars 3 forks source link

fix get-metric-data #74

Closed atheurer closed 2 years ago

atheurer commented 2 years ago

-broke after changing mgetPeriodRange, now expects 2D array (to organize by iteration, then by samples). getPeriodRange(), still used on some places, did not create 2D array from single period ID when calling mgetPeriodRange()

-fix a number of for loops which did not explicitly declare "i", resulting in i being potentially modified in calls to other functions within for loop.