You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2019/12/09 00:56:47 UTC

[GitHub] [incubator-mxnet] leezu commented on issue #17012: Upgrade 3rdparty/openmp to release_90 version

leezu commented on issue #17012: Upgrade 3rdparty/openmp to release_90 version
URL: https://github.com/apache/incubator-mxnet/pull/17012#issuecomment-563020783
 
 
   Please see the last line of the PR description above. Given that the assert is still there and it is not failing anymore I think we can establish that there is no bug in MXNet. Thus I can't follow your reasoning that there's a bug in MXNet.
   I think it's more likely that there was a bug in the 2 year old openmp version.
   
   Then please rescind your veto or give some evidence that there is a bug. Currently I consider this to be a veto without technical justification.
   
   Thanks! 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services