Re: [Olena-patches] r4432: Implement HQ2x upscaling algorithm
"FF" == Fabien Freling <fabien.freling@lrde.epita.fr> writes:
FF> URL: https://svn.lrde.epita.fr/svn/oln/trunk/milena/sandbox FF> ChangeLog: FF> 2009-09-04 Fabien Freling <fabien.freling@lrde.epita.fr> FF> Implement HQ2x upscaling algorithm. FF> * fabien/mln/upsampling/hq2x.hh: Implement HQ2x algorithm. FF> * fabien/mln/upsampling/hq3x.hh: Remove dead code. FF> * fabien/tests/upsampling/Makefile: Add `hq2x' target. FF> * fabien/tests/upsampling/hq2x.cc: New test file for HQ2x. Je suis assez choqué. Ce code est une adaptation d'un code de Maxim Stepin sous LGPL 2.1 : on reconnaît clairement la logique du code original, même si les accès à l'image ont changé. Cependant 1) le nom de l'auteur n'apparaît nul part 2) son copyright a été modifié en (c) LRDE 3) la license a été changée en GPL 2 + exception. Le dépôt SVN de milena est public. Tu ne peux pas prendre le code de quelqu'un et le redistribuer sous une autre license comme ça ! Et indépendamment du problème de copyright/license, quand on implémente un algo précis, c'est une bonne idée d'en donner les références (article, page web, ...) dans l'entête. FF> --- trunk/milena/sandbox/fabien/mln/upsampling/hq2x.hh (revision 0) FF> +++ trunk/milena/sandbox/fabien/mln/upsampling/hq2x.hh (revision 4432) FF> @@ -0,0 +1,2988 @@ FF> +// Copyright (C) 2009 EPITA Research and Development Laboratory (LRDE) FF> +// FF> +// This file is part of Olena. FF> +// FF> +// Olena is free software: you can redistribute it and/or modify it under FF> +// the terms of the GNU General Public License as published by the Free FF> +// Software Foundation, version 2 of the License. FF> +// FF> +// Olena is distributed in the hope that it will be useful, FF> +// but WITHOUT ANY WARRANTY; without even the implied warranty of FF> +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU FF> +// General Public License for more details. FF> +// FF> +// You should have received a copy of the GNU General Public License FF> +// along with Olena. If not, see <http://www.gnu.org/licenses/>. FF> +// FF> +// As a special exception, you may use this file as part of a free FF> +// software project without restriction. Specifically, if other files FF> +// instantiate templates or use macros or inline functions from this FF> +// file, or you compile this file and link it with other files to produce FF> +// an executable, this file does not by itself cause the resulting FF> +// executable to be covered by the GNU General Public License. This FF> +// exception does not however invalidate any other reasons why the FF> +// executable file might be covered by the GNU General Public License. -- Alexandre Duret-Lutz
Alexandre Duret-Lutz <adl@lrde.epita.fr> writes:
"FF" == Fabien Freling <fabien.freling@lrde.epita.fr> writes:
FF> URL: https://svn.lrde.epita.fr/svn/oln/trunk/milena/sandbox FF> ChangeLog: FF> 2009-09-04 Fabien Freling <fabien.freling@lrde.epita.fr>
FF> Implement HQ2x upscaling algorithm. FF> * fabien/mln/upsampling/hq2x.hh: Implement HQ2x algorithm. FF> * fabien/mln/upsampling/hq3x.hh: Remove dead code. FF> * fabien/tests/upsampling/Makefile: Add `hq2x' target. FF> * fabien/tests/upsampling/hq2x.cc: New test file for HQ2x.
Je suis assez choqué. Ce code est une adaptation d'un code de Maxim Stepin sous LGPL 2.1 : on reconnaît clairement la logique du code original, même si les accès à l'image ont changé. Cependant
1) le nom de l'auteur n'apparaît nul part 2) son copyright a été modifié en (c) LRDE 3) la license a été changée en GPL 2 + exception.
Le dépôt SVN de milena est public. Tu ne peux pas prendre le code de quelqu'un et le redistribuer sous une autre license comme ça !
J'appuie. Il faut faire très attention à ça. Lorsque l'on reprend du code, il faut laisser le copyright header à l'identique : ce n'est pas parce que tu modifies un fichier que tu en deviens légalement l'auteur. Et ça fait partie des avantages du libre : on peut panacher des bouts de code à condition de respecter le droit d'auteur et les termes de la licence.
Et indépendamment du problème de copyright/license, quand on implémente un algo précis, c'est une bonne idée d'en donner les références (article, page web, ...) dans l'entête.
Absolument ! Il faut penser aux autres (nous + les utilisateurs externes du projet). Roland
2009/9/7 Roland Levillain <roland@lrde.epita.fr>:
Alexandre Duret-Lutz <adl@lrde.epita.fr> writes:
"FF" == Fabien Freling <fabien.freling@lrde.epita.fr> writes:
FF> URL: https://svn.lrde.epita.fr/svn/oln/trunk/milena/sandbox FF> ChangeLog: FF> 2009-09-04 Fabien Freling <fabien.freling@lrde.epita.fr>
FF> Implement HQ2x upscaling algorithm. FF> * fabien/mln/upsampling/hq2x.hh: Implement HQ2x algorithm. FF> * fabien/mln/upsampling/hq3x.hh: Remove dead code. FF> * fabien/tests/upsampling/Makefile: Add `hq2x' target. FF> * fabien/tests/upsampling/hq2x.cc: New test file for HQ2x.
Je suis assez choqué. Ce code est une adaptation d'un code de Maxim Stepin sous LGPL 2.1 : on reconnaît clairement la logique du code original, même si les accès à l'image ont changé. Cependant
1) le nom de l'auteur n'apparaît nul part 2) son copyright a été modifié en (c) LRDE 3) la license a été changée en GPL 2 + exception.
Le dépôt SVN de milena est public. Tu ne peux pas prendre le code de quelqu'un et le redistribuer sous une autre license comme ça !
J'appuie. Il faut faire très attention à ça.
Lorsque l'on reprend du code, il faut laisser le copyright header à l'identique : ce n'est pas parce que tu modifies un fichier que tu en deviens légalement l'auteur.
Et ça fait partie des avantages du libre : on peut panacher des bouts de code à condition de respecter le droit d'auteur et les termes de la licence.
J'étais justement en train de reprendre les headers. Ce premier commit etait surtout histoire de sauvegarder le travail.
Et indépendamment du problème de copyright/license, quand on implémente un algo précis, c'est une bonne idée d'en donner les références (article, page web, ...) dans l'entête.
Absolument ! Il faut penser aux autres (nous + les utilisateurs externes du projet).
C'est prévu dans le prochain commit avec les références nécessaires. J'aurais du le faire au début, mea culpa. -- Fabien Freling Email: fabien.freling@gmail.com Website: http://www.ffreling.com Phone: +33 (0)6 07 08 24 84
Fabien Freling <fabien.freling@lrde.epita.fr> writes:
2009/9/7 Roland Levillain <roland@lrde.epita.fr>:
Alexandre Duret-Lutz <adl@lrde.epita.fr> writes:
"FF" == Fabien Freling <fabien.freling@lrde.epita.fr> writes:
FF> URL: https://svn.lrde.epita.fr/svn/oln/trunk/milena/sandbox FF> ChangeLog: FF> 2009-09-04 Fabien Freling <fabien.freling@lrde.epita.fr>
FF> Implement HQ2x upscaling algorithm. FF> * fabien/mln/upsampling/hq2x.hh: Implement HQ2x algorithm. FF> * fabien/mln/upsampling/hq3x.hh: Remove dead code. FF> * fabien/tests/upsampling/Makefile: Add `hq2x' target. FF> * fabien/tests/upsampling/hq2x.cc: New test file for HQ2x.
Je suis assez choqué. Ce code est une adaptation d'un code de Maxim Stepin sous LGPL 2.1 : on reconnaît clairement la logique du code original, même si les accès à l'image ont changé. Cependant
1) le nom de l'auteur n'apparaît nul part 2) son copyright a été modifié en (c) LRDE 3) la license a été changée en GPL 2 + exception.
Le dépôt SVN de milena est public. Tu ne peux pas prendre le code de quelqu'un et le redistribuer sous une autre license comme ça !
J'appuie. Il faut faire très attention à ça.
Lorsque l'on reprend du code, il faut laisser le copyright header à l'identique : ce n'est pas parce que tu modifies un fichier que tu en deviens légalement l'auteur.
Et ça fait partie des avantages du libre : on peut panacher des bouts de code à condition de respecter le droit d'auteur et les termes de la licence.
J'étais justement en train de reprendre les headers. Ce premier commit etait surtout histoire de sauvegarder le travail.
Je comprends. Mais comme le dépôt est public, il faut qu'on fasse attention avec ça. Je vais refaire encore un peu d'évangélisation pour Git ;) mais il s'agit là de l'un des cas d'utilisations que ce dernier sait bien gérer. Tu peux en effet créer un certain nombre de commits locaux, puis, lorsque tu es content du résultat, fusionner tout ou partie de ces commits, avant de les envoyer au serveur. Ainsi, tu peux masquer des étapes intermédiaires qui ne sont que le produit du processus de travail.
Et indépendamment du problème de copyright/license, quand on implémente un algo précis, c'est une bonne idée d'en donner les références (article, page web, ...) dans l'entête.
Absolument ! Il faut penser aux autres (nous + les utilisateurs externes du projet).
C'est prévu dans le prochain commit avec les références nécessaires. J'aurais du le faire au début, mea culpa.
OK ! Roland
participants (3)
-
Alexandre Duret-Lutz -
Fabien Freling -
Roland Levillain