Bug 116576

Summary:	UI, HELP: Wrong description of Function PHI: distribution function instead of density function
Product:	LibreOffice	Reporter:	Gilward Kukel <gilward.kukel>
Component:	Documentation	Assignee:	Rafael Lima <rafael.palma.lima>
Status:	RESOLVED FIXED
Severity:	normal	CC:	olivier.hallot, sophia.schroeder, xiscofauli
Priority:	medium
Version:	6.0.2.1 release
Hardware:	All
OS:	All
Whiteboard:	target:7.3.0
Crash report or crash signature:		Regression By:
Bug Depends on:
Bug Blocks:	120200

Description Gilward Kukel 2018-03-23 01:51:52 UTC

Description:
The description of the function PHI in the function wizard and in the help is wrong. It says that it returns the values of the distribution function for a standard normal distribution. But it is the density function.

https://helponline.libreoffice.org/latest/en-US/text/scalc/01/04060184.html?DbPAR=CALC#bm_id3152806

Actual Results:  
 

Expected Results:
 


Reproducible: Always


User Profile Reset: No



Additional Info:
See also:
http://docs.oasis-open.org/office/v1.2/os/OpenDocument-v1.2-os-part2.html#__RefHeading__1018744_715980110

https://support.office.com/en-us/article/phi-function-23e49bc6-a8e8-402d-98d3-9ded87f6295c?omkt=en-US&ui=en-US&rs=en-US&ad=US


User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:59.0) Gecko/20100101 Firefox/59.0

Comment 1 Mike Kaganski 2018-03-23 02:43:14 UTC

Confirming.

According to OASIS OpenDocument Version 1.2,

> PHI(N) is a synonym for NORMDIST(N,0,1,FALSE()).

> NORMDIST ... returns the value of the probability density function or the cumulative distribution function ...
>
> Syntax: NORMDIST( Number x ; Number Mean ; Number StandardDeviation [ ; Logical Cumulative = TRUE() ] )

When the Cumulative is FALSE() - as in PHI(N), - it returns the PDF; when TRUE(), it returns its integral (=CDF) - see https://en.wikipedia.org/wiki/Cumulative_distribution_function.

Gilward Kukel, thank you for the report; and you are welcome to submit a patch to correct this issue. This could be done online (in web UI) after you created an account at https://gerrit.libreoffice.org, and submitted a license statement according to https://wiki.documentfoundation.org/Development/Developers#License_Statements. This way, you can continue making corrections in the future as soon as you find an issue in documentation, have the corrections done immediately, and further contribute to benefit of all LibreOffice users.

Comment 2 Rafael Lima 2021-08-25 13:34:47 UTC

This bug still persists in LO 7.2 and 7.3 alpha.

https://help.libreoffice.org/latest/en-US/text/scalc/01/04060184.html?DbPAR=CALC#bm_id3152806

https://help.libreoffice.org/7.3/en-US/text/scalc/01/04060184.html?DbPAR=CALC#bm_id3152806

It's a simple fix... I'm taking it.

Comment 3 Rafael Lima 2021-08-26 14:08:23 UTC

A patch has been proposed in:
https://gerrit.libreoffice.org/c/help/+/121058

It is available for review.

Comment 4 Commit Notification 2021-08-27 18:43:51 UTC

Rafael Lima committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/help/commit/04f49b69b26dc5d9cb2939f86b7c10653b7c36e6

tdf#116576 Fix the description of the PHI function

Comment 5 Rafael Lima 2021-09-20 20:23:39 UTC

This bug is now fixed in the latest build of the help:

https://help.libreoffice.org/7.3/en-US/text/scalc/01/04060184.html?DbPAR=CALC#bm_id3152806