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

2019-09-11 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-530575110 Anyway seems that this PR has some fundamental flaws that are far beyond my intellect to comprehend, so I will go ahead and close

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

2019-09-11 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-530575127 Anyway seems that this PR has some fundamental flaws that are far beyond my intellect to comprehend, so I will go ahead and close

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

2019-09-11 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-530572078 Wasn't hacky a very bad slang according to the politburo? Maybe you can show how to make it better. For me it works for building

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

2019-09-09 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-529746151 I changed the build to release. Is there any other concrete changes you are requesting so we can get done with this? This has been

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

2019-09-09 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-529745539 https://www.python.org/doc/sunset-python-2/ This is an automated

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

2019-09-07 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-529167275 @apeforest you already approved this PR before. Why are you requesting additional changes again after some time? Is this to

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

2019-09-07 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-529167191 This PR was opened 5 months ago, adding comments ad infinitum after a few rounds of CR and existing approvals is not a good use of

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

2019-07-18 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-513069465 @aaronmarkham as per dev@ discussion, Python 2 will be deprecated soon. I would recommend only referring to Python3 in

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

2019-06-24 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-505180689 please suggest path forward: Shall python3 mention be removed? Shall we close this PR as it doesn't advance? @apeforest

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

2019-06-18 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-503288770 @aaronmarkham what do you suggest? to change python3 commands to just python ? I thought there was a discussion about dropping

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

2019-06-17 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-502900772 @vandanavk rebased This is an automated message from the Apache Git

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

2019-06-05 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-499284534 Any more changes requested? This is an automated message from the

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

2019-04-05 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-480438211 @apeforest can you review again? This is an automated message from

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

2019-04-01 Thread GitBox
larroy commented on issue #14535: [DOC] Updated install instructions for mac URL: https://github.com/apache/incubator-mxnet/pull/14535#issuecomment-478772429 @apeforest would you prefer something like ./dev_menu.py build? You can always use CMake to build. this is just a convenience