Jagadish Krishnamoorthy
Jagadish Krishnamoorthy
lgtm, approved.
ping @jithunnair-amd @jeffdaily @loadams
@loadams any comments on this PR?
> > LGTM, but let's add a unit test to ensure this functionality can be tested on ROCm (and CUDA) > > @jagadish-amd - thoughts on adding unit tests for...
> > > LGTM, but let's add a unit test to ensure this functionality can be tested on ROCm (and CUDA) > > > > > > @jagadish-amd - thoughts...
> @jagadish-amd please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information. > > ``` > @microsoft-github-policy-service agree [company="{your company}"] >...
Some of the tf32 test cases in test_nn are failing due to accuracy error. Resolving it and then will remove the DO NOT MERGE label.