One: [luci-interpreter] Unit testing TODOs

Created on 19 May 2020  路  5Comments  路  Source: Samsung/ONE

  • Add tests for KernelBuilder.
  • Check output shapes in kernel tests.
  • Improve checks for consistency of types and shapes in configure and execute.
  • Add negative kernel tests.
typtechnical debt

Most helpful comment

Basically speaking, it is the work which should have been done right away, but for some reasons has been posponed. Please check out the wikipedia article for more.
I used this label because the items in the list should be implemented, but they are not that critical to do it right now.

All 5 comments

What is type/technical debt and why did you add this label?

Basically speaking, it is the work which should have been done right away, but for some reasons has been posponed. Please check out the wikipedia article for more.
I used this label because the items in the list should be implemented, but they are not that critical to do it right now.

@s-barannikov Is it okay for you if I create functions in testutil for output tensor shape check?

@struss
Sure. I just have one wish: it would be nice to see what exactly differs in shapes in case a test fails.

@s-barannikov okay. I will make a draft PR for this. thanks a lot.

Was this page helpful?
0 / 5 - 0 ratings