bugGNU Octave - Bugs: bug #45866, [octave forge] (fuzzy-logic)...

 
 

bug #45866: [octave forge] (fuzzy-logic) next_row_index calculation incorrect in getfis_two_args in getfis.m

Submitter:  Brie Zobel <briezee>
Submitted:  Wed 02 Sep 2015 10:49:41 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  None Assigned to:  None
Originator Name:  Brie Z Open/Closed:  * Open
Release:  * other Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 03 Sep 2015 10:53:41 AM UTC, comment #1: 

Adding package maintainer to CC.

Carnë Draug <carandraug>
Group Member
Wed 02 Sep 2015 10:49:41 PM UTC, original submission:  

In the fuzzy-logic-toolkit, (version 0.4.5), in getfis.m, in method getfis_two_args, in the case statement for inmfparams,
the next_row_index calculation is



next_row_index = (i - 1) * max_len + j;


This however, uses the number of columns when attempting to calculate number of rows.

I believe this line should be


next_row_index = (i-1)*columns(fis.input(i).mf) +j;


This is line 344 of getfis.m, in the fuzzy-logic-toolkit, version 0.4.5.

Brie Zobel <briezee>

 

(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 jwe (Updated the item)
  • -email is unavailable- added by carandraug (Linda Markowsky - maintainer of the fuzzy-logic-toolkit package)
  • -email is unavailable- added by briezee (Submitted the item)
  • -email is unavailable- added by briezee
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-08-13 jwe Summaryfuzzy-logic-toolkit package: next_row_index calculation incorrect in getfis_two_args in getfis.m [octave forge] (fuzzy-logic) next_row_index calculation incorrect in getfis_two_args in getfis.m
    2015-09-03 carandraug Summarynext_row_index calculation incorrect in getfis_two_args in getfis.m fuzzy-logic-toolkit package: next_row_index calculation incorrect in getfis_two_args in getfis.m
        Carbon-Copy- Added -email is unavailable-
    2015-09-02 briezee Carbon-Copy- Added briezee

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code