[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-10-01 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-426089635 @iblis17 Hopefully one last thing - On the dev list we are thinking of submitting a request to Apache Infra to merge the PR. @marcoabreu had the idea

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-28 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-425597494 We're investigating a simple merge option now on the dev list. Sorry @iblis17 that it's taking so long to get in... we are working on it.

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-28 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-425567605 Unfortunately, we still can't merge through the web page. I brought this up on the dev list too, but I feel since the merging task is going to be a bit

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-28 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-425540855 Thanks @iblis17 - I'll keep a close watch on CI and try to push the button as soon as it finishes :)

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-28 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-425418802 Unfortunately, @mjwall was right in regards to the conflicts. I tried to merge this morning and got this ![import_julia_binding_by_iblis17_

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-27 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-425070133 Going to wait to merge 24 more hours to make sure we have consensus on the dev list about the best merge method to use

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-26 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-424905669 There is a discussion on the dev list about how best to merge

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-26 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-424697668 @marcoabreu can you please help setup a Confluence account for @iblis17 ? I'm not sure how. I'm going to merge this in 24 hours with "No Squash"

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-25 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-424420321 Does everyone feel comfortable merging this now? It would be good to get this large initial chunk of work in so other improvements can be made in a

[GitHub] gigasquid commented on issue #10149: import Julia binding

2018-09-23 Thread GitBox
gigasquid commented on issue #10149: import Julia binding URL: https://github.com/apache/incubator-mxnet/pull/10149#issuecomment-423837255 Can you please merge master into your branch and see if that fixes the CI failure? Once it is  we can see if we can get it merged