Olaf Lipinski
Olaf Lipinski
@cantonios I have added the split test, however it does not seem to pass, even on the release branch on my local install.
@cantonios whatever part was failing yesterday, it is fine now. I have also fixed the `testGradientsAll`, and now all `array_ops` test pass for me locally.
@cantonios thanks for looking at the issue here! I have fixed it, hopefully, in the last commit.
Seems like almost all tests succeeded except for one, but I am not sure what to make of this. I don't know if this PR is what breaks this? ```...
> @olipinski could you please rebase the branch to head , thank you! Done!
@cantonios I wanted `int8` support because I am using a dynamically generated dataset, which for decreasing memory usage uses `int8`. So, I think overall support for `int8` would be useful...
Other than the memory footprint, personally, I would not use it for anything else. A use-case that I could see is for quantised networks and embedded devices, but if the...
Sorry, I meant if it is not desirable to add it for a single use case of reducing memory consumption. I guess the main reason to do that would be...
@cantonios I have made the pull request - #57056
Is there anything else I need to do to complete this PR?