Qianqian Zhu
Qianqian Zhu
Regarding the first issue, `expected_speed_image1` is too small, I remember that we are considering remove the case or extend the speed. @ngu-niny Do we have any conclusion on this?
@rayx Thanks for reporting it, this test is out of date, we might abandon this x3130-upstream and xio3130-downstream structure, have something else replacing them. @jingzhao84 Do we plan to replace...
@jingzhao84 @huangyum Would you help check this? Thanks.
@leidwang Any other comments?
Hi @6-dehan and @nanliu-r Please help review, especially check if the test steps are expected. I have a basic test passed but it definitely require a full regression test. Thanks.
My test all passed, suppose there is still some issue in current logic, I will run more test to debug. Thanks @6-dehan @leidwang
@nanliu-r Would you please help review? Thanks.
@leidwang Is this still needed in this repo?