Ticket #55 (closed defect: fixed)

Opened 1 year ago

Last modified 2 months ago

correlation analysis

Reported by: willaume@phimeca.com Assigned to: regis.lebrun@eads.net
Priority: minor Milestone: openturns-0.13.0
Component: documentation Version: 0.11.1
Keywords: PCC, SRC, PRCC, SRRC Cc:

Description

I don't understand the use of the methods :

  • .PCC
  • .PRCC
  • .SCC
  • .SRCC

The first two methods give an error message which seems to be a bug. The second two methods give another error message. (see attached files)

Attachments

PCC.py (2.4 kB) - added by willaume@phimeca.com on 11/15/07 11:20:42.
PRCC.py (2.4 kB) - added by willaume@phimeca.com on 11/15/07 11:20:51.
SCC.py (2.4 kB) - added by willaume@phimeca.com on 11/15/07 11:20:59.
SRCC.py (2.4 kB) - added by willaume@phimeca.com on 11/15/07 11:21:08.
pdebuild.log (1.6 MB) - added by conty on 12/12/07 09:42:29.
Ubuntu Gutsy Gibbon amd64 compilation log file.

Change History

11/15/07 11:20:42 changed by willaume@phimeca.com

  • attachment PCC.py added.

11/15/07 11:20:51 changed by willaume@phimeca.com

  • attachment PRCC.py added.

11/15/07 11:20:59 changed by willaume@phimeca.com

  • attachment SCC.py added.

11/15/07 11:21:08 changed by willaume@phimeca.com

  • attachment SRCC.py added.

11/15/07 21:48:36 changed by anonymous

It follows from the definition of PCC and PRCC taht these quantities cannot be computed if the dimension of the input sample is not at least equal to 2, see OpenTURNS_ReferenceGuide, section C'. Nevertheless, OpenTURNS should complain about the wrong input sample dimension and it doesn't, which is a bug. It will be fixed in the next maintenance release and the next release.

For the two files SCC.py and SRCC.py, there is a mispelling between SCC (does not exist) and SRC (Standard Regression Coefficient), and between SRCC (does not exist) and SRRC (Standard Rank Regrassion Coefficient).

11/16/07 08:48:36 changed by willaume@phimeca.com 

For the two files SCC.py and SRCC.py, there is a mispelling between SCC
(does not exist) and SRC (Standard Regression Coefficient), and between
SRCC (does not exist) and SRRC (Standard Rank Regrassion Coefficient).

So there is a bug in the TUI user manual.

11/19/07 22:52:38 changed by regis.lebrun@eads.net

  • keywords set to PCC, SRC, PRCC, SRRC.
  • priority changed from unspecified to minor.
  • component changed from library to documentation.

Yes: there is a bug in the documentation. By the way, it is clearly stated in the exact same documentation that the dimension of the firt sample must be > 1 for PCC and PRCC ;-), and it is also (wrongly) stated that the same restriction occurs for SRC and SRRC. Nevertheless, I added a test to CorrelationAnalysis class and I requalify this ticket as a documentation issue.

11/22/07 11:33:58 changed by willaume@phimeca.com

In my documentation I have no sign ">", just a reverse "?".

(follow-ups: ↓ 6 ↓ 7 ) 11/24/07 15:37:19 changed by regis.lebrun@eads.net

Something bad during the LaTeX compilation? Can you give the pdflatex compilation log?

(in reply to: ↑ 5 ) 12/02/07 21:40:31 changed by conty

  • status changed from new to assigned.

Replying to regis.lebrun@eads.net:

Something bad during the LaTeX compilation? Can you give the pdflatex compilation log?

Users from Phimeca Engineering use Debian packages, they don't have any compilation logs. As maintener, I can deliver you the compilation log of building package process. Unfortunately I have removed all logs of 0.11.1 building process but the 0.11.2 is coming (very) soon and I will keep them.

(in reply to: ↑ 5 ) 12/12/07 09:39:27 changed by conty

  • owner changed from conty to regis.lebrun@eads.net.
  • status changed from assigned to new.

Ubuntu/Gutsy in amd64 compilation log files attached.

12/12/07 09:42:29 changed by conty

  • attachment pdebuild.log added.

Ubuntu Gutsy Gibbon amd64 compilation log file.

01/24/08 20:49:14 changed by conty

  • status changed from new to closed.
  • resolution set to fixed.
  • milestone set to openturns-0.11.3.

01/25/08 21:47:09 changed by anne.dutfoy@edf.fr

  • status changed from closed to reopened.
  • resolution deleted.

I fixed the typo in the documentation concerning these methods.

01/26/08 14:51:55 changed by regis.lebrun@eads.net

  • status changed from reopened to closed.
  • resolution set to fixed.

11/07/08 16:02:02 changed by willaume@phimeca.com

  • status changed from closed to reopened.
  • resolution deleted.

In the TUI user manual I downloaded a few minutes ago :

  • the sample 1 for PCC and PRCC methods is still specified of dimension greater or equal to 1.
  • SRCC is not corrected

11/07/08 22:55:19 changed by regis.lebrun@eads.net

  • status changed from reopened to closed.
  • resolution set to fixed.
  • milestone changed from openturns-0.11.3 to openturns-0.13.0.

This time, it is ok. I also updated the online documentation.

Régis