bugGNU Octave - Bugs: bug #56810, [octave forge] (nurbs)...

 
 

bug #56810: [octave forge] (nurbs) nrbbasisfunder fails when called with 4 output arguments

Submitted by:  Claudio Catterina <ccatterina>
Submitted on:  Mon 26 Aug 2019 05:44:45 PM UTC  
 
Category:  Octave Forge Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Unexpected Error
Status:  None Assigned to:  None
Originator Name:  Open/Closed:  Open
Release:  5.1.0 Operating System:  GNU/Linux

Add a New Comment (Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 27 Aug 2019 11:45:27 AM UTC, comment #2: 

Yeah, it seems to work properly with that change.

Claudio Catterina <ccatterina>
Tue 27 Aug 2019 10:49:08 AM UTC, comment #1: 

Yes, looks like a silly typo :-)

Have you tried if it works properly when changing
|| (nargout>3) ...
into
|| (nargout > 4) ...
?

cc'íng package maintainer

Philip Nienhuis <philipnienhuis>
Project Member
Mon 26 Aug 2019 05:44:45 PM UTC, original submission:  

As the function documentation says it should be possible to compute the basis functions derivatives with respect to the directions `u`, `v` and `w`:

%   [Bu, Bv, Bw, N] = nrbbasisfunder ({u, v, w}, vol)
%   [Bu, Bv, Bw, N] = nrbbasisfunder (pts, vol)

But due to the following check the function raise an error because nargout > 3:

  if (   (nargin<2) ...
      || (nargout>3) ...
      || (~isstruct(nrb)) ...
      || (iscell(points) && ~iscell(nrb.knots)) ...
      || (~iscell(points) && iscell(nrb.knots) && (size(points,1)~=numel(nrb.number))) ...
      || (~iscell(nrb.knots) && (nargout>2)) ...
      )
    error('Incorrect input arguments in nrbbasisfunder');
  end

Claudio Catterina <ccatterina>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by philipnienhuis
  • -email is unavailable- added by ccatterina (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

     

     

    Follows 1 latest change.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-08-27 philipnienhuis Carbon-Copy- => Added cdf

    Back to the top


    Powered by Savane 3.5