TheFoundryVisionmongers / nuke-ML-server

A Nuke client plug-in which connects to a Python server to allow Machine Learning inference in Nuke.
Apache License 2.0
134 stars 36 forks source link

Software license conflicts #13

Closed dekekincaid closed 5 years ago

dekekincaid commented 5 years ago

Hi Folks

The license for the project is listed as Apache License, Version 2.0. At the same time the readme and the MLClient node reads: "This is strictly non-commercial". These are legally in conflict. The Apache license you assigned to the code means I can pretty much do whatever I want with the code and sell it to whomever I want. Also the license is irrevocable.

Copied from the include Apache License 2.0 in this project:

  1. Grant of Copyright License. Subject to the terms and conditions of this License, each Contributor hereby grants to You a perpetual, worldwide, non-exclusive, no-charge, royalty-free, irrevocable copyright license to reproduce, prepare Derivative Works of, publicly display, publicly perform, sublicense, and distribute the Work and such Derivative Works in Source or Object form.

You really should remove the "This is strictly non-commercial" part from the readme and the node.

samhodge commented 5 years ago

Even the COCO dataset it was trained on is CC3.0

ringdk commented 5 years ago

Hello!

Thanks both for bringing that to our attention. We don't want to impose any restrictions or constraints on the Apache license. We also think the current licensing is a bit confusing, where it's all Apache except for some files relating to the 'DensePose' model under CC non-commercial.

To make things clearer we've removed all 'strictly non-commercial' mentions, as well as the 'DensePose' model to ensure everything is covered by the unmodified Apache 2.0 license.

Thanks again!