You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2020/07/22 06:05:47 UTC

[GitHub] [incubator-mxnet] ciyongch commented on issue #17329: License issue with 1.6.0.rc1

ciyongch commented on issue #17329:
URL: https://github.com/apache/incubator-mxnet/issues/17329#issuecomment-662260853


   Hi @szha, the way of handling dual license header was passed with lazy consensus, and I updated the related files based on the following conclusion.
   
   >Including the BSD-3 style license in releases wouldn't be a problem, as it's compatible with Apache License 2. As there are >substantial changes, I believe PPMC would prefer to put the ASF header on top of the file (ie. 2 headers) [72 hours lazy >consensus if there are no concerns]. We still need to declare all the numpy einsum derived files in the LICENSE and fix the >inconsistency that ASF header was removed in src/operator/numpy/np_einsum_op-inl.h but remains in >src/operator/numpy/np_einsum_path_op-inl.h 
   
   For more details, please see https://lists.apache.org/thread.html/r6d8f62b1ad34f6dbe109a01e715723e8996cd478a01a08498aa02a2e%40%3Cdev.mxnet.apache.org%3E


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org