so the original email suggesting to remove was after all self-serving :) let's encourage if someone wants to maintain and make use of the original work and make it better.
-1 to remove at this point P.S: I suggest to do some due diligence before bringing topics up for discussion. On Wed, Sep 11, 2019 at 8:10 AM Lv, Tao A <tao.a...@intel.com> wrote: > Sorry to chime in. > > There is a PR to fix amalgamation. I was pinged several times to merge it > but I don't think I have enough knowledge to do that. So it would be great > if someone from this thread can help to review. > > https://github.com/apache/incubator-mxnet/pull/15303 > > thanks, > -tao > > -----Original Message----- > From: Marco de Abreu <marco.g.ab...@gmail.com> > Sent: Wednesday, September 11, 2019 9:38 PM > To: dev@mxnet.incubator.apache.org > Subject: Re: [DISCUSS] Remove amalgamation > > Is Amalgamation only used on Android though? Are there any other use cases? > > -Marco > > Pedro Larroy <pedro.larroy.li...@gmail.com> schrieb am Mi., 11. Sep. 2019, > 11:57: > > > Hi Anirudh > > > > Appreciate your feedback and sorry if my email came across that way to > > you, I think you might miss some context. I don't think calling > > something hacky is anything bad and isn't supposed to be the topic of > > the discussion. It was reported as not working by users, hence the > > original thread. It was a request for opinions from people who might > > actually have tried to work in Mxnet on Android. > > > > Thanks. > > > > Pedro. > > > > > > On Tuesday, September 10, 2019, Anirudh Subramanian > > <anirudh2...@gmail.com > > > > > wrote: > > > Hi Pedro, > > > > > > I don't see anything "destructive" with Chris asking for > > > justification > > for > > > you calling something "hacky". The only email in this thread where I > > > see > > ad > > > hominems and disrespectful comments is your email. > > > > > > On Sat, Sep 7, 2019, 10:18 PM Pedro Larroy > > > <pedro.larroy.li...@gmail.com > > > > > > wrote: > > > > > >> Apache mentors should have a look at these reincident harassment > > >> and destructive behaviors which demotivate contributions and take > > >> action. It takes only one bad apple to ruin a community. > > >> > > >> The mobile solution that is known to work as of know is cross > > >> compiling with "ci/build.py -p build.android_armv8" or > > >> "build.android_armv7". The only advantage of amalgamation is to > > >> provide a smaller binary that we > > could > > >> accomplish with the C preprocessor. > > >> > > >> My technical contributions speak for themselves, including porting > > >> MXNet > > to > > >> Android and ARM and helping many users run MXNet in Jetson, > > >> Raspberry Pi and Android amongst many other topics. I have never > > >> been disrespectful > > to > > >> anyone. I'm entitled to my own technical opinions about > > >> amalgamation or > > any > > >> other piece of code whatsoever, that's no personal disrespect to > > >> anyone > > and > > >> perfectly valid. If you are not interested in this project anymore, > > >> do > > us > > >> all a favor and stop trolling and being toxic. If you want my > > >> respect, > > step > > >> up your technical contributions, be positive and encourage others, > > >> this including commits, I haven't seen for many months, please be > > >> positive > > and > > >> constructive. This scorched-earth attitude is only reflecting bad > > >> on > > you. > > >> I'm certainly not interested in your ad-hominems or unasked for > > technical > > >> advice, which to be honest, showing poor judgment and ignorance. > > >> Myself and others have come up with numbers, graphs, metrics and > > >> arguments and have been met with dismissal, trolling and > > >> sea-lioning. I have recieved your insults via public and private > > >> channels (such as linkedin) as have others. This is not ok and has > > >> to stop. If you have something personal against me or against your > > >> former employer, this is not the right place > > or > > >> forum. > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> > > >> On Fri, Sep 6, 2019 at 3:56 PM Chris Olivier > > >> <cjolivie...@gmail.com> > > >> wrote: > > >> > > >> > Hi Pedro, > > >> > > > >> > While I was not involved with amalgamation or its development in > > >> > any > > way, > > >> > can you please refrain from referring to the work of others as a > > "hacky > > >> > solution"? This is derogatory slang and the statement was not > > supported > > >> > with any justification for such name-calling. Someone spent a > > >> > good > > deal > > >> of > > >> > time on this solution at some point in time and I am sure it > > >> > worked > > for > > >> its > > >> > purpose at that time -- I think it was used in the original > > >> > javascript > > >> port > > >> > as well, actually -- and it is disrespectful to call their > > >> > efforts "hacky". Please respect what came before. > > >> > > > >> > Thanks for understanding, > > >> > > > >> > -Chris > > >> > > > >> > > > >> > On Fri, Sep 6, 2019 at 3:07 PM Pedro Larroy < > > >> pedro.larroy.li...@gmail.com> > > >> > wrote: > > >> > > > >> > > Hi > > >> > > > > >> > > I would like to propose to remove amalgamation from MXNet and > > >> > > CI, > > users > > >> > > have reported that they couldn't use it successfully in > > >> > > Android, and instead they were able to use the cross compiled > > >> > > docker build > > >> > successfully. > > >> > > > > >> > > Any reason why we shouldn't remove this hacky solution? > > >> > > > > >> > > Pedro. > > >> > > > > >> > > > >> > > > > > >