* Carnë Draug <carandraug+...@gmail.com> [2012-11-11 13:34]:

Fix with r11422. Can I further help with this by creating a proper COPYRIGHT file for the package? I already tried to follow the debian upstream guide http://wiki.debian.org/UpstreamGuide#Licenses that asks to list each file and its license.

This is not really necessary, but may help. When putting together Debian packages, We use a tool called licencecheck (in package devscripts) that parses every upstream file and automatically generate a copyright file according to DEP5 (http://dep.debian.net/deps/dep5/). This file is further edited for reducing the number of stanzas. I am attaching below the current debian/copyright file of the Debian package, which gets installed as /usr/share/doc/octave-image/copyright.

On 11 November 2012 09:54, Rafael Laboissiere <raf...@laboissiere.net> wrote:

3) The function fftconv2 gives a wrong result when called like this: "fftconv2 (vector1, vector2, matrix)", although this usage is advertised in the help string. For the Debian package, I am planning to apply the patch attached below, which (i) removes the advertisement from the help string, (ii) gives an error message when the function is called with the said usage and (iii) drop the tests with the said usage.

You forgot to attach the patch.

Sorry.  Attached now.

I don't know what changed to make this usage stop working so I only left a note on the source code and should not appear on the help string (at least it doesn't on my system). Do not remove the tests, mark them with xtest instead. I think I have done that before but apparently forgot to commit. I'd give a warning instead of an error since the reason it fails may be due to changes in the other functions it uses and users with different versions may actually get the correct results. Please see r11421.

Well, IMHO it is not a good idea to allow function to give results that are known to be wrong. I will see with the other members of the DOG (Debian Octave Group) how we should proceed while waiting for the fix from upstream.

Thanks,

Rafael

Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
Upstream-Name: Image package for Octave
Upstream-Contact: The Octave Community <octave-...@lists.sf.net>
Source: http://octave.sourceforge.net/image/

Files: *
Copyright: 1999-2001 Kai Habel <kai.ha...@gmx.de>
           1999, 2003 Andy Adler <ad...@sce.carleton.ca>
           2000 Etienne Grossmann <etie...@egdn.net>
           2000 Teemu Ikonen <tpiko...@pcu.helsinki.fi>
           2000 Paul Kienzle <pkien...@users.sf.net>
           2002 Jeff Orchard <j...@cs.sfu.ca>
           2002 Jeffrey E. Boyd <b...@cpsc.ucalgary.ca>
           2004 Josep Mones i Teixidor <jmo...@puntbarra.com>
           2004 Antti Niemistö <antti.niemi...@tut.fi>
           2004 Stefan van der Walt <ste...@sun.ac.za>
           2004, 2005, Justus H. Piater <justus.pia...@ulg.ac.be>
           2005 Carvalho-Mariel
           2005-2008, 2010 Søren Hauberg <so...@hauberg.org>
           2007 Alexander Barth <barth.alexan...@gmail.com>
           2008 Jonas Wagner <j....@gmx.ch>
           2006, 2009 Stefan Gustavson <stefan.gustav...@gmail.com>
           2009 Roderick Koehle <koe...@users.sourceforge.net>
           2009 Sergey Kirgizov
           2010 Alex Opie <lx...@orcon.net.nz>
           2010 Andrew Kelly, IPS Radio & Space Services
           2010-2012 Carnë Draug <carandraug+...@gmail.com>
           2011 Adrián del Pino <delpinonavarr...@gmail.com>
           2011 William Krekeler <wkreke...@cleanearthtech.com>
           2011 Santiago Reyes González <mailalcu...@gmail.com>
           2011, 2012 Jordi Gutiérrez Hermoso <jord...@octave.org>
           2012 Adam H Aitkenhead <adamaitkenh...@hotmail.com>
           2012 Rik Wehbring
           2012 Pablo Rossi <pro...@ing.unrc.edu.ar>
License: GPL-3+

Files: ./inst/immaximas.m
Copyright: 2003-2005, Peter Kovesi
License: MIT/X11 (BSD like)
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
 in the Software without restriction, subject to the following conditions:
 .
 The above copyright notice and this permission notice shall be included in all
 copies or substantial portions of the Software.
 . 
 The Software is provided "as is", without warranty of any kind.

Files: inst/bmpwrite.m inst/colfilt.m inst/colorgradient.m
Copyright: Paul Kienzle <pkien...@users.sf.net>
License: Public domain
 This code is in the public domain.

Files: inst/fftconv2.m src/graycomatrix.cc src/hough_line.cc
Copyright: 2004, Stefan van der Walt <ste...@sun.ac.za>
License: BSD-2
 This program is free software; 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 AUTHOR 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 AUTHOR 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.

Files: debian/*
Copyright: 2008 Ólafur Jens Sigurðsson <ojs...@gmail.com>
           2008-2009, 2012 Rafael Laboissiere <raf...@laboissiere.net>
           2008-2011 Thomas Weber <twe...@debian.org>
License: GPL-3+

License: GPL-3+
 This program is free software; you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
 the Free Software Foundation; either version 3 of the License, or
 (at your option) any later version.
 .
 This program is distributed in the hope that it will be useful,
 but WITHOUT ANY WARRANTY; without even the implied warranty of
 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 GNU General Public License for more details.
 .
 You should have received a copy of the GNU General Public License
 along with this program; if not, see <http://www.gnu.org/licenses/>.
 .
 On Debian systems, the complete text of the GNU General Public
 License, version 3, can be found in the file
 `/usr/share/common-licenses/GPL-3'.
Description: Drop usage of fftconv2 with two vectors and one matrix as arguments
Author: Rafael Laboissiere <raf...@laboissiere.net>
Last-Update: 2012-11-11

--- octave-image-2.0.0.orig/inst/fftconv2.m
+++ octave-image-2.0.0/inst/fftconv2.m
@@ -23,7 +23,6 @@
 
 ## -*- texinfo -*-
 ## @deftypefn {Function File} fftconv2 (@var{a}, @var{b}, @var{shape})
-## @deftypefnx{Function File} fftconv2 (@var{v1}, @var{v2}, @var{a}, @var{shape})
 ## Convolve 2 dimensional signals using the FFT.
 ##
 ## This method is faster but less accurate than @var{conv2} for large @var{a} and @var{b}.
@@ -46,12 +45,8 @@ function X = fftconv2(varargin)
 
         ## usage: fftconv2(v1, v2, a[, shape])
 
-        rowcolumn = 1;
-        v1 = varargin{1}(:)';
-        v2 = varargin{2}(:);
-        orig_a = varargin{3};
+       error ("fftconv2: usage 'fftconv2(v1, v2, a[, shape])' is not yet implemented");
 
-        if (nargin == 4) shape = varargin{4}; endif
     else
         ## usage: fftconv2(a, b[, shape])
 
@@ -106,13 +101,6 @@ endfunction
 %!assert(norm(fftconv2(b,a,'valid')-conv2(b,a,'valid')), 0, 1e6*eps)
 
 %!# usage: fftconv2(v1, v2, a[, shape])
-%!##FIXME this usage seems to be broken...
-%!shared x,y,a
-%! x = 1:4; y = 4:-1:1; a = repmat(1:10, 5);
-%!assert(norm(fftconv2(x,y,a)-conv2(x,y,a)), 0, 1e6*eps)
-%!assert(norm(fftconv2(x,y,a,'full')-conv2(x,y,a,'full')), 0, 1e6*eps)
-%!assert(norm(fftconv2(x,y,a,'same')-conv2(x,y,a,'same')), 0, 1e6*eps)
-%!assert(norm(fftconv2(x,y,a,'valid')-conv2(x,y,a,'valid')), 0, 1e6*eps)
 
 %!demo
 %! ## Draw a cross
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_nov
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to