-
Notifications
You must be signed in to change notification settings - Fork 222
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test case code refactoring #134
Comments
@Knight-X The only test cases that truly need IDX files are the tensor ops tests and IDX loader. These are assumed to be fixed mathematical operators with well defined behavior as a function of their inputs. |
@Knight-X @mbartling @dboyliao |
rom_tensor is fine. But we still need a tool to generate weight array or we have to do it manually. Compared to pushing weight files on github, I think that generating the necessary weight files before running all tests make more sense. |
In order to simplify the steps of writing test case, we probably need to do some refactoring. For example, we need to run tensorflow and generate idx file manually now. I think that we can write scripts to generate data before running test case automatically.
Then, we can just compare the result between tensorflow and utensor. It will be easier to extend other test case to increase test coverage.
The text was updated successfully, but these errors were encountered: