bugGNU Octave - Bugs: bug #63019, [octave forge] (linear-algebra)...

 
 

bug #63019: [octave forge] (linear-algebra) nmf_pg document minor error

Submitter:  Liang Tang <lt1234>
Submitted:  Mon 05 Sep 2022 12:56:11 PM UTC
 
Category:  Octave Forge Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Documentation
Status:  Confirmed Assigned to:  None
Originator Name:  lt1234 Open/Closed:  * Open
Release:  * other Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

Mon 05 Sep 2022 01:18:37 PM UTC, comment #2: 

Oops. Didn't realize this is a documentation bug.

Re-opening

Markus Mützel <mmuetzel>
Project Member
Mon 05 Sep 2022 01:16:42 PM UTC, comment #1: 

IIUC, you expect `nmf_pg(A,tol=1e-3);` to behave like a Python idiom.
However, in Octave, it means: Assign `1e-3` to the variable `tol` and pass that result (as the second argument) to the function `nmf_pg`.
IIUC, that function expects you to pass property-value pairs instead (like you showed later on in your report). So, you see that error message.

Closing as invalid.

Markus Mützel <mmuetzel>
Project Member
Mon 05 Sep 2022 12:56:11 PM UTC, original submission:  

In linear algebra package, the command line help of nmf_pg shows:

Examples:

            A     = rand(10,5);
            [W H] = nmf_pg(A,tol=1e-3);
            U     = W*H -A;
            disp(max(abs(U)));

These error messages would show:

error: nmf_pg: non-string for Parameter name or Switch
error: called from
    error at line 592 column 7
    parse at line 480 column 11
    nmf_pg at line 79 column 3

These work:

            [W H] = nmf_pg(A);
            [W H] = nmf_pg(A,'tol', 1e-3);

Liang Tang <lt1234>

 

(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 mmuetzel (Posted a comment)
  • -email is unavailable- added by lt1234 (Submitted the item)
  •  

    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 project members can vote.

     

     

     

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-09-05 mmuetzel StatusInvalid Confirmed
        Open/ClosedClosed Open
        Release7.1.0 other
        Operating SystemMicrosoft Windows Any
        Summarynmf_pg document minor error [octave forge] (linear-algebra) nmf_pg document minor error
    2022-09-05 mmuetzel StatusNone Invalid
        Open/ClosedOpen Closed

    Back to the top


    Powered by Savane 3.9