szha commented on issue #13964: Addresses comments in runtime feature discovery 
API
URL: https://github.com/apache/incubator-mxnet/pull/13964#issuecomment-459112043
 
 
   since the information is fixed after compilation, libinfo sounds more 
accurate. there's no need to maintain the consistency in naming between module 
name of python frontend and the file name in the backend.
   
   On the API design, yes, I feel strongly about the change, hence the change 
request. The reason is that the library information doesn't change after it's 
compiled, and since it's loaded only once, there's no futher need to query the 
backend, especially given that ctypes isn't exactly fast. Thus, having an API 
that queries all feature status at the beginning, and make that information 
available should be the goal. To achieve that goal, a single method with 
complete information should be the way to go.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to