search

sccn / amica

Code for AMICA: Adaptive Mixture ICA with shared components
BSD 2-Clause "Simplified" License
23 stars 13 forks source link

LICENSE #6

Closed co60ca closed 3 years ago

co60ca commented 7 years ago

Hey Jason,

We use Amica a lot over here at Brock University and I just wanted to thank you for you efforts.

Just was going through and was packaging some of your binaries and noticed you don't provide a license. GPL3 would be best or MIT. EEGLAB is LGPL so GPL might be a good fit anyway.

Thanks,

palday commented 7 years ago

Indeed, a lack of license is really problematic, see e.g. this. Publishing the code is certainly a nice step, but a tiny one. :/

palday commented 3 years ago

@japalmer29 bump on this.

co60ca commented 3 years ago

Hi from 3 years in the future, maybe an email would be a good idea haha. I have no stake in Amica anymore so I can't devote time to following up.

japalmer29 commented 3 years ago

Amica is included with EEGLAB. So the license is the same as that for EEGLAB. rhe Python library MNE I think is BSD and there seems to be a limiting conflict. Currently there is no solution to this issue.

Jason

On Thu, Oct 22, 2020, 4:37 AM Maeve Kennedy notifications@github.com wrote:

Hi from 3 years in the future, maybe an email would be a good idea haha

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/japalmer29/amica/issues/6#issuecomment-713828640, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACRBESTGTYKZDSYSOXQSET3SL4Z6JANCNFSM4DW6SVRA .

palday commented 3 years ago

Since this hosted in a different repository, it would be great to have a license notice here. Also note that the copyright holder can issue multiple licenses -- so having both a GPL and a BSD license would be possible. Also licensing AMICA under only BSD or MIT would still allow bundling in a GPL package like EEGLAB.

arnodelorme commented 3 years ago

EEGLAB is BSD not GPL. Also even if it was GPL the bundling would be possible.

On Oct 21, 2020, at 12:07 PM, Phillip Alday notifications@github.com wrote:

Since this hosted in a different repository, it would be great to have a license notice here. Also note that the copyright holder can issue multiple licenses -- so having both a GPL and a BSD license would be possible. Also licensing AMICA under only BSD or MIT would still allow bundling in a GPL package like EEGLAB.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

palday commented 3 years ago

Ah, yes, I missed the license change, but I did say bundling would be possible.

Nonetheless, an explicit declaration of the AMICA license in the AMICA repository would be great, even at the level of "This source file is part of EEGLAB".

co60ca commented 3 years ago

Copy it over into LICENSE ? I'd submit a pull request but I don't know who the copyright holder technically is.

Copyright <YEAR> <COPYRIGHT HOLDER>

Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions are met:

1. Redistributions of source code must retain the above copyright notice,
this list of conditions and the following disclaimer.

2. Redistributions in binary form must reproduce the above copyright notice,
this list of conditions and the following disclaimer in the documentation
and/or other materials provided with the distribution.

THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF
THE POSSIBILITY OF SUCH DAMAGE.
japalmer29 commented 3 years ago

Oh, I guess there was a change I forgot about from the original license.

I’m not sure if I am the copyright holder, or if there are restrictions from UCSD.

I am fine with adding the BSD license statement.

Arno, could you or one of the other contributors add the license statement?

Thanks,

Jason

From: Arnaud Delorme [mailto:notifications@github.com] Sent: Wednesday, October 21, 2020 3:47 PM To: japalmer29/amica amica@noreply.github.com Cc: Jason Palmer japalmer29@gmail.com; Mention mention@noreply.github.com Subject: Re: [japalmer29/amica] LICENSE (#6)

EEGLAB is BSD not GPL. Also even if it was GPL the bundling would be possible.

On Oct 21, 2020, at 12:07 PM, Phillip Alday <notifications@github.com mailto:notifications@github.com > wrote:

Since this hosted in a different repository, it would be great to have a license notice here. Also note that the copyright holder can issue multiple licenses -- so having both a GPL and a BSD license would be possible. Also licensing AMICA under only BSD or MIT would still allow bundling in a GPL package like EEGLAB.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/japalmer29/amica/issues/6#issuecomment-713921611 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACRBESUA3IRTL3JXKRL3EU3SL5QIDANCNFSM4DW6SVRA . https://github.com/notifications/beacon/ACRBESVSMZNTFFCJTBSTNFDSL5QIDA5CNFSM4DW6SVRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFKGZISY.gif

ericphanson commented 3 years ago

Bump :)

japalmer29 commented 3 years ago

Sorry for the lack of clarity on license. Amica is released under BSD license, like eeglab.

On Mon, Feb 1, 2021, 9:53 PM Eric Hanson notifications@github.com wrote:

Bump :)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/japalmer29/amica/issues/6#issuecomment-770835149, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACRBESUGGACH3PITK7FSUJDS42P5NANCNFSM4DW6SVRA .

jadonogh commented 3 years ago

Bump :), hopefully Phillip’s pr closes it! @arnodelorme @japalmer29

arnodelorme commented 3 years ago

I will leave it up to Jason. Jason you can accept the pull request if you agree with the BSD license terms.

On Mar 23, 2021, at 7:05 PM, Jacob Donoghue @.***> wrote:

Bump :), hopefully Phillip’s pr closes it! @arnodelorme @japalmer29

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

japalmer29 commented 3 years ago

Sorry I thought this was done already. I merged the pr.

Jason

From: Arnaud Delorme @. Sent: Wednesday, March 24, 2021 3:08 PM To: japalmer29/amica @.> Cc: Jason Palmer @.>; Mention @.> Subject: Re: [japalmer29/amica] LICENSE (#6)

I will leave it up to Jason. Jason you can accept the pull request if you agree with the BSD license terms.

On Mar 23, 2021, at 7:05 PM, Jacob Donoghue @. <mailto:@.> > wrote:

Bump :), hopefully Phillip’s pr closes it! @arnodelorme @japalmer29

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/japalmer29/amica/issues/6#issuecomment-805528699 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ACRBESU2HS4F22X7JGTGTVLTFF6SLANCNFSM4DW6SVRA . https://github.com/notifications/beacon/ACRBESXI2D6ZM3ZUNEG72ADTFF6SLA5CNFSM4DW6SVRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGABWI6Y.gif