bugGNU Octave - Bugs: bug #64328, [octave-forge](statistics) provide...

 
 

bug #64328: [octave-forge](statistics) provide default parameters for logi (logistic) distribution

Submitter:  Muhali <muhali>
Submitted:  Tue 20 Jun 2023 06:34:54 PM UTC
   
 
Category:  Octave Package Severity:  1 - Wish
Priority:  5 - Normal Item Group:  Feature Request
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 8.1.0 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 24 Mar 2024 01:01:53 PM UTC, comment #5: 

Closing report as fixed

Nicholas Jankowski <nrjank>
Group Member
Sat 23 Mar 2024 10:26:41 PM UTC, comment #4: 

Since the implementation of distribution objects in statistics-1.6.4, the 'makedist' function returns the distribution objects with the default parameter values. So even if a user would like to use default parameters with logi* functions and does not know what the default parameter should be, there is an easy way to figure this out. I think this thread can be closed. Thanks

Andreas Bertsatos <pr0m1th3as>
Fri 26 Jan 2024 12:01:42 AM UTC, comment #3: 

The reason logipdf and a couple of other functions were changed so that they require all parameters than having the defaults, was an attempt to have the same interface with all other functions. And since most dist functions did not use default values, I decided to remove the default parameters from the rest with the exception of norm* functions (due to standard normal). I understand that this may not appeal to some users and may also break some older code, but it was a decision of preference (mostly based on my experience teaching statistics) to urge students to get familiar with distribution parameters. This was the main reason why all dist functions got a link to the corresponding wikipedia page as well. However, thank you for pointing out the inconsistency in the help docstring of logipdf function, which I fixed with this commit https://github.com/gnu-octave/statistics/commit/76c1236f88caa1e4dbc612411c1c8df1f27a930c
I've also fixed a few other inconsistencies in the documentation of the distribution functions. This can be closed.

Andreas Bertsatos <pr0m1th3as>
Mon 10 Jul 2023 12:11:37 AM UTC, comment #2: 

just noticed this report.  might be best to file it over on the statistics package github issue tracker:
https://github.com/gnu-octave/statistics/issues

(did cc maintainer though.)

Nicholas Jankowski <nrjank>
Group Member
Wed 21 Jun 2023 07:05:25 PM UTC, comment #1: 

I'm not the maintainer of the statistic package, but I agree that this would be a good idea.  The various probability density functions should have the same interface which includes being able to call the function with just one input parameter (data).  Also, the documentation says that logipdf can be called in this manner. 


 -- statistics: Y = logipdf (X)
 -- statistics: Y = logipdf (X, MU)
 -- statistics: Y = logipdf (X, MU, S)

     Logistic probability density function (PDF).


So, if this isn't fixed then the documentation needs to be changed as well.

Rik <rik5>
Group administrator
Tue 20 Jun 2023 06:34:54 PM UTC, original submission:  

To allow greater compatibility of logi to the older logistic_ distribution I suggest to provide default values of 0 and 1 for mu and sigma, similar to other distributions (such as norm).

Muhali <muhali>

 

(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 pr0m1th3as (Posted a comment)
  • -email is unavailable- added by nrjank (Posted a comment)
  • -email is unavailable- added by nrjank
  • -email is unavailable- added by muhali (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 group members can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2024-03-24 nrjank StatusConfirmed Fixed
        Open/ClosedOpen Closed
    2023-07-10 nrjank Carbon-Copy- Added pr0m1th3as
    2023-06-21 rik5 Carbon-CopyRemoved 72865 -
    2023-06-21 rik5 Item GroupNone Feature Request
        StatusNone Confirmed
    2023-06-21 rik5 CategoryNone Octave Package
        Severity3 - Normal 1 - Wish

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code