ANSHUMAN87 commented on a change in pull request #7267:
URL: https://github.com/apache/tvm/pull/7267#discussion_r562744658



##########
File path: python/tvm/relay/frontend/tensorflow.py
##########
@@ -942,7 +942,10 @@ def _impl(inputs, attr, params, mod):
         )
 
         if sparse_lhs:
-            data = _op.transpose(data)
+            if attr.get("adjoint_a"):

Review comment:
       Yes i agree the logic is not straight forward, however it is to match 
the operation towards Topi Op.
   I think matching series of If may not be good as it will branch out complete 
code if we want to tag with each case.
   But i can try putting some comment over the current code, so that it will 
help to understand logic better. Please let me know if you think otherwise. 




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to