[RESULTS] [VOTE] add conan support for Apache MXNet (incubating)

2019-05-07 Thread Konstantin Ivlev
Dear MXNet community, I'm happy to announce the results of the vote. This vote passes with 0 +1 votes (0 binding), no 0 votes, and 2 -1 vote. +1 votes * No votes 0 votes * No votes -1 votes * kellen sunderland * Sheng Zha Vote thread can be found here [1]. The list of members can be found here

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-06 Thread Sheng Zha
Hi Konstantin, Thanks for your reply. > I personally prefer small incremental changes, and starting with some MVP, > where M is > minimal, meaning least possible effort (e.g. number of dependencies) used > from conan. Agreed. That said, this seems like a case where adoption decision can't be b

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-06 Thread Konstantin Ivlev
Hi Sheng Zha, > Currently, the linked PR only includes OpenBLAS actually, it's OpenBLAS + OpenCV + lapack, three libraries as proof-of-concept > A proof-of-concept that shows it actually replaces more dependency than openblas would be helpful may we define an actual scope then, e.g. how much dep

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-04 Thread Sheng Zha
To be clear, my intention is really to prevent a seemingly good solution to exacerbate the problem that it sets out to solve. This tends to happen when there are not enough people to drive it to the end. If there are additional values in this solution that people feel outweighs the problems bel

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-04 Thread Sheng Zha
Thank you for the explanation and sorry that I missed the earlier context as it has been a while. While I like the idea of simplifying the dependency management with tools like conan, I have the following concerns on this vote as-is (it's also my take on why I think the PR is stuck): - It's unc

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-03 Thread Junru Shao
Hi Konstantin, Kellen, Thank you guys for the very detailed explanation! I was lacking some relevant contexts and previous discussions, which got me confused previously. My understanding that C++ is short of a perfect package manager. I totally agree that for any C++ project, it would be great to

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-03 Thread Konstantin Ivlev
Hi Junru, > I am actually a bit concerned about the security issues. We are asked to download binaries from third-party websites, which are not controlled or validated by Apache it's possible to run conan server inside apache network and download binaries and sources only from this remote > CMake

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-03 Thread kellen sunderland
So firstly let's try to keep our responses empathetic and avoid ad-hom comments. It might be beneficial to take some time to review the Apache Code of Conduct [1]. Konstantin has taken a lot of time to think about dependency management in MXNet on a volunteer basis which is commendable. Second,

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-03 Thread Junru Shao
I am actually a bit concerned about the security issues. We are asked to download binaries from third-party websites, which are not controlled or validated by Apache. Although it is claimed to be “decentralized”, I am really not convinced where the security comes from. In the meantime, sacrificing

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-03 Thread kellen sunderland
Hey Konstantin. Thanks for starting an email thread and sorry for the confusion. I think the ides is that we should discuss and agree on Conan.io adoption first on the dev list, then start merging PRs. Release 1.4.1 is already in testing and the 1.5 code freeze deadline is also near so I think i

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-03 Thread Konstantin Ivlev
hi Sheng Zha, on pull request review I was told by Anirudh anirudhacharya and Roshani Nagmote to start discussion/vote on the mxnet dev list. it seems to be a vicious circle now - on GitHub I am told to use vote, and on vote I am told to use GitHub, this doesn't help much. FYI GitHub review stuck,

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-03 Thread Konstantin Ivlev
hi Sheng Zha, on pull request review I was told by Anirudh anirudhacharya and Roshani Nagmote to start discussion/vote on the mxnet dev list. it seems to be a vicious circle now - on GitHub I am told to use vote, and on vote I am told to use GitHub, this doesn't help much. FYI GitHub review stuck,

Re: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-03 Thread Sheng Zha
Hi Konstantin, While conan looks like an option that's worth exploring, given that your request is to merge the pull request, I'd suggest that the request should go through the regular pull request review and it doesn't really need a vote (as it doesn't substitute reviews anyway) If you would

[VOTE] add conan support for Apache MXNet (incubating)

2019-05-02 Thread Konstantin Ivlev
Dear MXNet community, This is the 3-day vote to add conan support for Apache MXNet (incubating) version v1.4.1. The voting on dev@ list will start May 03 23:59:59 (PST) and close on May 06 23:59:59. Background: conan is open-source, freeware, cross-platform package manager for C and C++ projects,

Fwd: [VOTE] add conan support for Apache MXNet (incubating)

2019-05-02 Thread Konstantin Ivlev
Dear MXNet community, This is the 3-day vote to add conan support for Apache MXNet (incubating) version v1.4.1. The voting on dev@ list will start May 03 23:59:59 (PST) and close on May 06 23:59:59. Background: conan is open-source, freeware, cross-platform package manager for C and C++ projects,