Re: [CANCELLED][VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-07-03 Thread Sergio Fernández
Exactly, 1.2.2 addressing the warning issue. Specially since 1.3.0 is being already discussed at dev@. Anyway, the podling can learn from these things. On Tue, Jul 3, 2018, 00:34 Justin Mclean wrote: > Hi, > > > Agree with Dave: incubating releases are not cheap… > > Also agree. Seems to me

Re: [CANCELLED][VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-07-03 Thread Justin Mclean
Hi, > Agree with Dave: incubating releases are not cheap… Also agree. Seems to me like something that could be improved on in a later release - release early and often! [1] Thanks, Justin 1. https://en.wikipedia.org/wiki/Release_early,_release_often

Re: [CANCELLED][VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-07-02 Thread Sergio Fernández
Agree with Dave: incubating releases are not cheap... On Mon, Jul 2, 2018, 20:02 Anirudh wrote: > Hi Dave, > > Thank you for your inputs. To give more context, the reason we are doing > the release is to fix a backwards incompatible change with the save_params > API. This issue impacted some of

Re: [CANCELLED][VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-07-02 Thread Anirudh
Hi Dave, Thank you for your inputs. To give more context, the reason we are doing the release is to fix a backwards incompatible change with the save_params API. This issue impacted some of our users and the warning message in RC0 showed an alternative API for one use case but not the other when

Re: [CANCELLED][VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-07-02 Thread Dave Fisher
Mentors all - Releases are not cheap and we are all volunteers. Can we teach Release Managers to have a little more judgement about when to pull a release? It seems to be too risk adverse to cancel this vote due to a warning message. My 2 cents. Regards, Dave Sent from my iPhone > On Jul 2,

[CANCELLED][VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-07-02 Thread Anirudh
Hi, We are cancelling this vote because we need to improve the warning message when users use a deprecated API: `save_params`. We will address the issue and send out a new release for another vote. Thanks, Anirudh

Re: [VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-06-28 Thread Hen
+1. I reviewed the diff between this release and the last. Hen On Fri, Jun 22, 2018 at 12:15 PM, Anirudh wrote: > Hi all, > > This is a call for releasing Apache MXNet (incubating) 1.2.1, release > candidate RC0. > > Apache MXNet (incubating) community has voted and approved the release. >

Re: [VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-06-25 Thread Anirudh
Hi Sergio, The 1.2.0 release notes have been added to NEWS.md. Thanks for pointing about CODEOWNERS file. Agree that it is not required in the source release and I will add its removal to the release steps so that it can be removed next time. Anirudh On Sun, Jun 24, 2018 at 6:45 PM, Sergio

Re: [VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-06-24 Thread Sergio Fernández
+1 (binding) So far I've checked: * signatures and digests * source releases file layouts * matched git tags and commit ids * incubator suffix and disclaimer * LICENSE & NOTICE files look good * no binary files in the source release * clean build (osx 10.12.6) Some minor details I'd like to

Re: [VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-06-23 Thread Sebastian
+1 (binding) Tested checksum, signature and compilation on ubuntu 16.04. -sebastian On 22.06.2018 21:15, Anirudh wrote: Hi all, This is a call for releasing Apache MXNet (incubating) 1.2.1, release candidate RC0. Apache MXNet (incubating) community has voted and approved the release. Vote

[VOTE] Apache MXNet (incubating) 1.2.1 release RC0

2018-06-22 Thread Anirudh
Hi all, This is a call for releasing Apache MXNet (incubating) 1.2.1, release candidate RC0. Apache MXNet (incubating) community has voted and approved the release. Vote thread: https://lists.apache.org/thread.html/52a6487bc1389d8865ccc0e42ade49