dmknutsen
dmknutsen
@dzbaker Kequan may be referring to Fix #2381 It was not merged due to potential downstream impacts on performance. We are still setting up the lab to better characterize performance...
I think the latter makes sense...but can't say I have a strong opinion either way. The one thing I would caution is that this change would also likely require requirements...
That works for me!
@dallen-osr Good catch and thanks for pointing this out! Was this caught through manual inspection or did you use a tool to catch the error? If a tool was used...where...
Apologies - just noticed I missed this. The test failure can not be repeated - this is a very infrequent occurrence...and requires execution of hundreds of tests to occur. Further,...
Yep - we could just put the updates in the example_platform_cfg.h (and change the name as part of the test setup). I'm a bit surprised that the functional tests failed,...
Good observation/point. I reached back to the project that originally requested standardizing the command responses to get further clarification on this. In this case - we increment the command counter...
Option 3 is best. The information conveyed in the messages is useful to the end user. If projects have bandwidth limitations and/or wish to reduce the message flow, they can...