search

mlcommons / power-dev

Dev repo for power measurement for the MLPerf™ benchmarks
https://mlcommons.org/en/groups/best-practices-power
Apache License 2.0
16 stars 24 forks source link

checker fails "Check client server messages" #243

Closed trevor-cockrell closed 3 years ago

trevor-cockrell commented 3 years ago

On master branch, commit e329e92f.

Checker output fails with:

[ ] Check client server messages                                                                     
        Client commands list length (9) should be less than server commands list length (9) by one.

The output is identical between the two messages (server,client). Both are:

[{'cmd': 'mlcommons/power client v3',
'cmd_time': 1624463804.767848,
'reply': 'mlcommons/power server v3',
'reply_time': 1624463804.767848},
{'cmd': 'time',
'cmd_time': 1624463804.7775543,
'reply': '1624463804.7775543',
'reply_time': 1624463804.7775543},
{'cmd': 'new,rnnt_Server_default,a96a049d-bd66-42a5-aa1c-b7386e740b13',
'cmd_time': 1624463804.7775543,
'reply': 'OK 2021-06-23_10-56-44_rnnt_Server_default,cdd3cd28-b5d0-461d-838f-49f423c90aea',
'reply_time': 1624463804.7775543},
{'cmd': 'time',
'cmd_time': 1624463804.7775543,
'reply': '1624463804.7775543',
'reply_time': 1624463804.7775543},
{'cmd': 'session,2021-06-23_10-56-44_rnnt_Server_default,start,ranging',
'cmd_time': 1624463804.7775543,
'reply': 'OK',
'reply_time': 1624463829.5544128},
{'cmd': 'session,2021-06-23_10-56-44_rnnt_Server_default,stop,ranging',
'cmd_time': 1624463937.4251857,
'reply': 'OK',
'reply_time': 1624463947.4551477},
{'cmd': 'time',
'cmd_time': 1624463947.4754834,
'reply': '1624463947.4754834',
'reply_time': 1624463947.4754834},
{'cmd': 'session,2021-06-23_10-56-44_rnnt_Server_default,start,testing',
'cmd_time': 1624463947.4754834,
'reply': 'OK',
'reply_time': 1624463960.612304},
{'cmd': 'session,2021-06-23_10-56-44_rnnt_Server_default,stop,testing',
'cmd_time': 1624464058.3049333,
'reply': 'OK',
'reply_time': 1624464068.3379998}]

Is there a reason that the client should be one less? Why would it not be correctly populating as one less command in that case?

trevor-cockrell commented 3 years ago

no longer an issue with checker updates