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 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
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>


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
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



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 our users and the warning message in RC0
> showed an alternative API for one use case but not the other when
> save_params was called. We are fixing it to provide a warning message for
> both use cases, so that users may not end up replacing save_params with the
> wrong API.
> My sincere apologies to the mentors, for creating additional work.
>
> Anirudh
>
> On Mon, Jul 2, 2018, 6:44 PM Dave Fisher  wrote:
>
> > 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, 2018, at 5:45 PM, Anirudh  wrote:
> > >
> > > 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
> >
> >
> > -
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
> >
>


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
save_params was called. We are fixing it to provide a warning message for
both use cases, so that users may not end up replacing save_params with the
wrong API.
My sincere apologies to the mentors, for creating additional work.

Anirudh

On Mon, Jul 2, 2018, 6:44 PM Dave Fisher  wrote:

> 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, 2018, at 5:45 PM, Anirudh  wrote:
> >
> > 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
>
>
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>


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, 2018, at 5:45 PM, Anirudh  wrote:
> 
> 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


-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



[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.
>
> Vote thread:
>
> https://lists.apache.org/thread.html/52a6487bc1389d8865ccc0e42ade49
> edde02c061090389616c8088d5@%3Cdev.mxnet.apache.org%3E
>
> Results thread:
>
> https://lists.apache.org/thread.html/1d833bc94897132590950668eaff7d
> 7e253894e2c274b1223ef733e1@%3Cdev.mxnet.apache.org%3E
>
> The source tarball, including signatures, digests etc. can be found at:
>
> https://dist.apache.org/repos/dist/dev/incubator/mxnet/1.2.1.rc0/
>
> The release tag can be found here:
>
> https://github.com/apache/incubator-mxnet/releases/tag/1.2.1.rc0
>
> 
> The release hash is daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8 and can be
> found here:
>
> https://github.com/apache/incubator-mxnet/commit/
> daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8
>
> Release artifacts are signed with the following key:
>
>
> 0D04 3D0F 92CA 7333 22A8  D131 9067 CC8E AF55 CD6D
>
>
> KEYS file available:
>
> https://dist.apache.org/repos/dist/dev/incubator/mxnet/KEYS
>
> The vote will end at Tuesday June 26th, 12:15 PDT.
>
> [ ] +1 Release this package as 1.2.0
> [ ] +0 no opinion
> [ ] -1 Do not release this package because ...
>
> Anirudh
>


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 Fernández  wrote:

> +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 comment:
> * I don't understand what's the purposed of CODEOWNERS file in the source
> release
> * I've noticed the patch release is not included in the README's changelog.
>
> On Fri, Jun 22, 2018 at 12:16 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.
> >
> > Vote thread:
> >
> > https://lists.apache.org/thread.html/52a6487bc1389d8865ccc0e42ade49
> > edde02c061090389616c8088d5@%3Cdev.mxnet.apache.org%3E
> >
> > Results thread:
> >
> > https://lists.apache.org/thread.html/1d833bc94897132590950668eaff7d
> > 7e253894e2c274b1223ef733e1@%3Cdev.mxnet.apache.org%3E
> >
> > The source tarball, including signatures, digests etc. can be found at:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/mxnet/1.2.1.rc0/
> >
> > The release tag can be found here:
> >
> > https://github.com/apache/incubator-mxnet/releases/tag/1.2.1.rc0
> >
> > 
> > The release hash is daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8 and can be
> > found here:
> >
> > https://github.com/apache/incubator-mxnet/commit/
> > daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8
> >
> > Release artifacts are signed with the following key:
> >
> >
> > 0D04 3D0F 92CA 7333 22A8  D131 9067 CC8E AF55 CD6D
> >
> >
> > KEYS file available:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/mxnet/KEYS
> >
> > The vote will end at Tuesday June 26th, 12:15 PDT.
> >
> > [ ] +1 Release this package as 1.2.0
> > [ ] +0 no opinion
> > [ ] -1 Do not release this package because ...
> >
> > Anirudh
> >
>


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 comment:
* I don't understand what's the purposed of CODEOWNERS file in the source
release
* I've noticed the patch release is not included in the README's changelog.

On Fri, Jun 22, 2018 at 12:16 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.
>
> Vote thread:
>
> https://lists.apache.org/thread.html/52a6487bc1389d8865ccc0e42ade49
> edde02c061090389616c8088d5@%3Cdev.mxnet.apache.org%3E
>
> Results thread:
>
> https://lists.apache.org/thread.html/1d833bc94897132590950668eaff7d
> 7e253894e2c274b1223ef733e1@%3Cdev.mxnet.apache.org%3E
>
> The source tarball, including signatures, digests etc. can be found at:
>
> https://dist.apache.org/repos/dist/dev/incubator/mxnet/1.2.1.rc0/
>
> The release tag can be found here:
>
> https://github.com/apache/incubator-mxnet/releases/tag/1.2.1.rc0
>
> 
> The release hash is daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8 and can be
> found here:
>
> https://github.com/apache/incubator-mxnet/commit/
> daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8
>
> Release artifacts are signed with the following key:
>
>
> 0D04 3D0F 92CA 7333 22A8  D131 9067 CC8E AF55 CD6D
>
>
> KEYS file available:
>
> https://dist.apache.org/repos/dist/dev/incubator/mxnet/KEYS
>
> The vote will end at Tuesday June 26th, 12:15 PDT.
>
> [ ] +1 Release this package as 1.2.0
> [ ] +0 no opinion
> [ ] -1 Do not release this package because ...
>
> Anirudh
>


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 thread:

https://lists.apache.org/thread.html/52a6487bc1389d8865ccc0e42ade49
edde02c061090389616c8088d5@%3Cdev.mxnet.apache.org%3E

Results thread:

https://lists.apache.org/thread.html/1d833bc94897132590950668eaff7d
7e253894e2c274b1223ef733e1@%3Cdev.mxnet.apache.org%3E

The source tarball, including signatures, digests etc. can be found at:

https://dist.apache.org/repos/dist/dev/incubator/mxnet/1.2.1.rc0/

The release tag can be found here:

https://github.com/apache/incubator-mxnet/releases/tag/1.2.1.rc0


The release hash is daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8 and can be
found here:

https://github.com/apache/incubator-mxnet/commit/
daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8

Release artifacts are signed with the following key:


0D04 3D0F 92CA 7333 22A8  D131 9067 CC8E AF55 CD6D


KEYS file available:

https://dist.apache.org/repos/dist/dev/incubator/mxnet/KEYS

The vote will end at Tuesday June 26th, 12:15 PDT.

[ ] +1 Release this package as 1.2.0
[ ] +0 no opinion
[ ] -1 Do not release this package because ...

Anirudh



-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



[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
edde02c061090389616c8088d5@%3Cdev.mxnet.apache.org%3E

Results thread:

https://lists.apache.org/thread.html/1d833bc94897132590950668eaff7d
7e253894e2c274b1223ef733e1@%3Cdev.mxnet.apache.org%3E

The source tarball, including signatures, digests etc. can be found at:

https://dist.apache.org/repos/dist/dev/incubator/mxnet/1.2.1.rc0/

The release tag can be found here:

https://github.com/apache/incubator-mxnet/releases/tag/1.2.1.rc0


The release hash is daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8 and can be
found here:

https://github.com/apache/incubator-mxnet/commit/
daa957c9c811b1abd09b98e8fb5ff2c06cad9ad8

Release artifacts are signed with the following key:


0D04 3D0F 92CA 7333 22A8  D131 9067 CC8E AF55 CD6D


KEYS file available:

https://dist.apache.org/repos/dist/dev/incubator/mxnet/KEYS

The vote will end at Tuesday June 26th, 12:15 PDT.

[ ] +1 Release this package as 1.2.0
[ ] +0 no opinion
[ ] -1 Do not release this package because ...

Anirudh