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/01/08 18:54:20 UTC

[GitHub] apeforest commented on issue #13795: add extra header file to include

apeforest commented on issue #13795: add extra header file to include
URL: https://github.com/apache/incubator-mxnet/pull/13795#issuecomment-452410955
 
 
   @eric-haibin-lin @szha The PR that requires this change is https://github.com/ctcyang/horovod/pull/19. The reason we need this header is that we are using `check_call` in Horovod to check the return status [here](https://github.com/ctcyang/horovod/pull/19/files#diff-6626ba53afb59b3f75cb017aa6259fd1R260). We need a way to catch the exception thrown from MXNet C API like [this](https://github.com/ctcyang/horovod/pull/19/files#diff-79b78539e15db1d75d73029490eed708R325). However, the `API_BEGIN()` and `API_END()` are defined in `c_api_common.h` which is the one I need to export. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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