Hi Niketan, I think it's probably safest to include the tensorflow and caffe licenses in all the artifacts that contain the generated files (the main jar and all the artifacts that use the main jar).
Deron On Mon, Apr 24, 2017 at 1:48 PM, Niketan Pansare <npan...@us.ibm.com> wrote: > Hi Deron, > > I had same doubt and hence have created the PR https://github.com/apache/ > incubator-systemml/pull/467 few days ago. We can continue to discuss this > on mailing list or the PR. > > Even though we are not including the caffe/tensorflow project, we are > depending on their input/output format (i.e. proto files). Unfortunately, > both caffe and tensorflow don't have maven coordinates for their > input/output formats as they tend to use C++/Python wrappers created from > the proto files. Hence, we have to compile their proto files into java > classes. Personally, I think we should include their licenses into our jar. > > TensorFlow => Apache 2.0 license > Caffe => https://github.com/BVLC/caffe/blob/master/LICENSE > > Thanks, > > Niketan Pansare > IBM Almaden Research Center > E-mail: npansar At us.ibm.com > http://researcher.watson.ibm.com/researcher/view.php?person=us-npansar > > [image: Inactive hide details for Deron Eriksson ---04/24/2017 01:28:58 > PM---Hi, I see after a recent commit that the main jar file now]Deron > Eriksson ---04/24/2017 01:28:58 PM---Hi, I see after a recent commit that > the main jar file now gets caffe and > > From: Deron Eriksson <deroneriks...@gmail.com> > To: dev@systemml.incubator.apache.org > Date: 04/24/2017 01:28 PM > Subject: caffe and org.tensorflow licenses? > ------------------------------ > > > > Hi, > > I see after a recent commit that the main jar file now gets caffe and > org.tensorflow packages added to it. This would tend to suggest that our > main jar needs modifications to its LICENSE file (and any other artifacts > containing the main jar would also need their LICENSES updated). > > However, I see at https://github.com/google/protobuf/blob/master/LICENSE > that: > "Code generated by the Protocol Buffer compiler is owned by the owner > of the input file used when generating it. This code is not > standalone and requires a support library to be linked with it. This > support library is itself covered by the above license." > > So could someone tell me if caffe and tensorflow licenses now need to be > added to the project? > > Even if they are not required, some confusion during artifact validation > can occur since the project looks like it contains caffe and tensorflow > classes. > > Deron > > > -- > Deron Eriksson > Spark Technology Center > http://www.spark.tc/ > > > > -- Deron Eriksson Spark Technology Center http://www.spark.tc/
