[GitHub] [incubator-mxnet] apeforest commented on issue #14535: [DOC] Updated install instructions for mac

[GitBox <gi...@apache.org>]

apeforest commented on issue #14535: [DOC] Updated install instructions for mac
URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-529246006
 
 
   My earlier approval was based on your assumption that python2 is no longer supported by mxnet. However, reading the latest discussion on devlist made me realize it is not the case. 
   
   If so, such change in your PR:
   `brew install python3` would seem not in sync with mxnet community decision.
   
   Also, this man page is crucial to any new and existing mxnet users, it's never too much to be cautious. I have also found the dev_menu.py usage very confusing. What's the benefit of adding another layer of yaml file on top of exiting CMakeList.txt? It has recently caused some issue for a new developer who build mxnet with this script without realizing the build type is set to Debug by default.
   
   In my opinion, we should teach users to change CMakeList.txt and build mxnet for different platforms.

----------------------------------------------------------------
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