[DISCUSS] Remove amalgamation
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 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 > 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 >> 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. >> > > >> > >> >
Re: [DISCUSS] Remove amalgamation
Is Amalgamation only used on Android though? Are there any other use cases? -Marco Pedro Larroy 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 > > 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 > > > 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 > >> 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. > >> > > > >> > > >> > > >
RE: [DISCUSS] Remove amalgamation
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 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 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 > > > 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 > > > > > 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 > >> > >> 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. > >> > > > >> > > >> > > >
