You are viewing a plain text version of this content. The canonical link for it is here.
Posted to general@incubator.apache.org by Serge Panev <sp...@nvidia.com.INVALID> on 2021/07/24 18:09:49 UTC
[VOTE] Release Apache MXNet (incubating) version 1.9.0.rc6
Dear community,
This is a call for a releasing Apache MXNet (incubating) 1.9.0, release
candidate 6.
Apache MXNet (incubating) community has voted and approved the release.
Vote thread:
https://lists.apache.org/thread.html/r2dc18e14551bd9cad66cdf3802de2fbbfb3dfe715e37a67a1bfc0a51%40%3Cdev.mxnet.apache.org%3E
Result thread:
https://lists.apache.org/thread.html/r6dc9556ad3cbde6708008e71683d71ebb5785cd5221db65db6959bc7%40%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.9.0.rc6/
The tag to be voted upon is 1.9.0.rc6:
https://github.com/apache/incubator-mxnet/releases/tag/1.9.0.rc6
The release hash is 20c4717e672101ca8b4a6e4017bdd7d438b58443:
https://github.com/apache/incubator-mxnet/commit/20c4717e672101ca8b4a6e4017bdd7d438b58443
KEYS file available:
https://dist.apache.org/repos/dist/dev/incubator/mxnet/KEYS
For information about the contents of this release, see:
https://cwiki.apache.org/confluence/display/MXNET/1.9.0+Release+Notes
The vote will be open for 7 days.
[ ] +1 release this package as 1.9.0.rc6
[ ] +0 no opinion
[ ] -1 do not release this package because...
Best regards,
Serge Panev
Re: [VOTE] Release Apache MXNet (incubating) version 1.9.0.rc6
Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,
Is this case the author / copyright owner is probably MATLAB / Math Works as that is where the code was copied from.
Kind Regards,
Justin
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org
Re: [VOTE] Release Apache MXNet (incubating) version 1.9.0.rc6
Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,
> With respect to your comments on the `erfinv` header file [1], the file references [2] as its source which is licensed under the new BSD license here [3].
Just because someone release a bundle of source under a license doesn’t change the of license terms of 3rd party code included in that bundle. Nor does its use by other people change the license.
I would suggest that you either:
a) Contact the author and ask permission to use the code / have it relicensed.
b) Replace the code with something under a compatible license.
Kind Regards,
Justin
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org
Re: [VOTE] Release Apache MXNet (incubating) version 1.9.0.rc6
Posted by Manu Seth <ma...@apache.org>.
Hi Justin,
Thanks for pointing out the licensing issues. We are working on addressing them.
With respect to your comments on the `erfinv` header file [1], the file references [2] as its source which is licensed under the new BSD license here [3]. There is a reference to MATLAB v2 which got released in 1986 and the function has since been adapted by several libraries including scipy (also under new BSD license), so I don't think we should be concerned about MATLAB license. Do you think referring to [3] in the header file [1] would suffice? Would like to know your thoughts on the same.
[1] https://github.com/apache/incubator-mxnet/blob/master/src/operator/contrib/erfinv-inl.h
[2] https://github.com/antelopeusersgroup/antelope_contrib/blob/master/lib/location/libgenloc/erfinv.c
[3] https://github.com/antelopeusersgroup/antelope_contrib/blob/master/lib/location/libgenloc/LICENSE
Thanks!
Manu Seth
On 2021/07/30 07:15:13, Justin Mclean <ju...@classsoftware.com> wrote:
> Hi,
>
> -1 (binding) for header and licensing issues
>
> There also seems to be an issue with the directories dlpack, dmlc, mshadow ands nnvm under the include directory in the tar file?
>
> I checked:
> - incubating in name
> - DISCLAIMER exists
> - LICENSE has issues
> - NOTICE seems OK
> - A lot of files have header issues (see below)
> - No unexpected binary files
>
> I did a spot check on LICENSE and found a number of issues. I did not check everything so there are likely to be more issue that those below.
> - This file [5] seem to incorrectly have an ASF header but is listed at MIT in the file and in LICENSE.
> - As do these files [6][7][8]
> - This file has a comment that states "Code is a direct translation of the erfinv m file in matlab version 2.0.” and the GitHub reference doesn't have a BSD file header. As far as I’m aware Matlab is not open source software. Is this Category X software?
> - This file seem to also incorrectly have an ASF header and is an unknown license. I can’t see mentioned in LICENSE.
> - This files seem to have incorrect licensing information [13] At the URL referenced it has a different header. I can’t see why the ASF copyright would be in this header.
> - These files seem to incorrectly have an ASF header [15][16][17]
>
> A large number of file have ASF headers but also state "Copyright (c) 201X by Contributors”. e.g [1][2][3][4] There looks to be more than 500 files like this. Are these 3rd party file or ASF files? If they are ASF files there there should be no copyright line.
>
> A number of files have "Copyright (c) 201X Microsoft” and also have ASF headers. Are these files 3rd party files or ASF files? e.g. [11] They look like 3rd party files to me. Some are Apache licensed but some are not. e.g [12] I cannot see this mentioned in LICENSE.
>
> I also noticed that the docker file refreshes to https://github.com/dmlc/mxnet Why is this? In fact there seems to be a large number of references to "https://github.com/dmlc” and "https://github.com/dmlc/mxnet".
>
> Kind Regards,
> Justin
>
> 1. ./include/mxnet/op_attr_types.h
> 2 ./cpp-package/include/mxnet-cpp/op_suppl.h
> 3. ./src/initialize.cc
> 4 ./src/operator/cudnn_bilinear_sampler-inl.h
> 5. ./src/operator/nn/layer_norm.cc
> 6. ./src/operator/numpy/np_einsum_op.cc
> 7. ./src/operator/numpy/np_einsum_op-inl.h
> 8. ./src/operator/numpy/np_einsum_path_op-inl.h
> 9 ./src/operator/contrib/erfinv-inl.h
> 10. ./example/image-classification/predict-cpp/image-classification-predict.cc
> 11. ./src/operator/contrib/deformable_psroi_pooling.cu
> 12. ./src/operator/contrib/modulated_deformable_convolution.cu
> 13 ./src/operator/contrib/erfinv-inl.h
> 14. https://github.com/numpy/numpy/blob/master/numpy/core/src/multiarray/einsum.c.src
> 15. vi ./python/mxnet/onnx/mx2onnx/_export_onnx.py
> 16 ./cmake/Modules/FindJeMalloc.cmake
> 17 ./src/operator/nn/mkldnn/mkldnn_base-inl.h
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
> For additional commands, e-mail: general-help@incubator.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org
Re: [VOTE] Release Apache MXNet (incubating) version 1.9.0.rc6
Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,
-1 (binding) for header and licensing issues
There also seems to be an issue with the directories dlpack, dmlc, mshadow ands nnvm under the include directory in the tar file?
I checked:
- incubating in name
- DISCLAIMER exists
- LICENSE has issues
- NOTICE seems OK
- A lot of files have header issues (see below)
- No unexpected binary files
I did a spot check on LICENSE and found a number of issues. I did not check everything so there are likely to be more issue that those below.
- This file [5] seem to incorrectly have an ASF header but is listed at MIT in the file and in LICENSE.
- As do these files [6][7][8]
- This file has a comment that states "Code is a direct translation of the erfinv m file in matlab version 2.0.” and the GitHub reference doesn't have a BSD file header. As far as I’m aware Matlab is not open source software. Is this Category X software?
- This file seem to also incorrectly have an ASF header and is an unknown license. I can’t see mentioned in LICENSE.
- This files seem to have incorrect licensing information [13] At the URL referenced it has a different header. I can’t see why the ASF copyright would be in this header.
- These files seem to incorrectly have an ASF header [15][16][17]
A large number of file have ASF headers but also state "Copyright (c) 201X by Contributors”. e.g [1][2][3][4] There looks to be more than 500 files like this. Are these 3rd party file or ASF files? If they are ASF files there there should be no copyright line.
A number of files have "Copyright (c) 201X Microsoft” and also have ASF headers. Are these files 3rd party files or ASF files? e.g. [11] They look like 3rd party files to me. Some are Apache licensed but some are not. e.g [12] I cannot see this mentioned in LICENSE.
I also noticed that the docker file refreshes to https://github.com/dmlc/mxnet Why is this? In fact there seems to be a large number of references to "https://github.com/dmlc” and "https://github.com/dmlc/mxnet".
Kind Regards,
Justin
1. ./include/mxnet/op_attr_types.h
2 ./cpp-package/include/mxnet-cpp/op_suppl.h
3. ./src/initialize.cc
4 ./src/operator/cudnn_bilinear_sampler-inl.h
5. ./src/operator/nn/layer_norm.cc
6. ./src/operator/numpy/np_einsum_op.cc
7. ./src/operator/numpy/np_einsum_op-inl.h
8. ./src/operator/numpy/np_einsum_path_op-inl.h
9 ./src/operator/contrib/erfinv-inl.h
10. ./example/image-classification/predict-cpp/image-classification-predict.cc
11. ./src/operator/contrib/deformable_psroi_pooling.cu
12. ./src/operator/contrib/modulated_deformable_convolution.cu
13 ./src/operator/contrib/erfinv-inl.h
14. https://github.com/numpy/numpy/blob/master/numpy/core/src/multiarray/einsum.c.src
15. vi ./python/mxnet/onnx/mx2onnx/_export_onnx.py
16 ./cmake/Modules/FindJeMalloc.cmake
17 ./src/operator/nn/mkldnn/mkldnn_base-inl.h
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscribe@incubator.apache.org
For additional commands, e-mail: general-help@incubator.apache.org