Opened 2 years ago

Closed 2 years ago

#2352 closed enhancement (done)

Correlation matrix should only show correlations to 3 or 4 digits accuracy

Reported by: gkronber Owned by: gkronber
Priority: medium Milestone: HeuristicLab 3.3.12
Component: Analysis.Statistics.Views Version: 3.3.11
Keywords: Cc:

Description

This is usually sufficient and the information can be represented more compactly this way

Change History (12)

comment:1 Changed 2 years ago by gkronber

  • Owner set to gkronber
  • Status changed from new to accepted

comment:2 Changed 2 years ago by gkronber

r12137: only show correlations to 3 digits accuracy.

comment:3 Changed 2 years ago by gkronber

I also had a look at a way to rotate the column headers of the correlation matrix by 90° (see http://stackoverflow.com/questions/5782674/net-winforms-vertical-text-in-datagridview) But the proposed solution is rather hacky and does not work as expected.

comment:4 Changed 2 years ago by gkronber

  • Owner changed from gkronber to mkommend
  • Status changed from accepted to reviewing

comment:5 Changed 2 years ago by mkommend

  • Owner changed from mkommend to gkronber

r12151: Reverse merged r12137 and implemented a format pattern for the EnhancedStringConvertibleMatrixView.

comment:6 Changed 2 years ago by gkronber

Ok, thanks but please reintroduce

} else if (i == j) { 	 	 
  dt[i, j] = 1.0; 
} ...

in CorrelationView.cs

comment:7 Changed 2 years ago by gkronber

  • Owner changed from gkronber to mkommend
  • Status changed from reviewing to assigned

comment:8 Changed 2 years ago by mkommend

  • Owner changed from mkommend to ascheibe
  • Status changed from assigned to reviewing

r12152: Readded line in CorrelationView (handling of diagonal values).

comment:9 Changed 2 years ago by ascheibe

  • Owner changed from ascheibe to mkommend

Ok, thanks.

comment:10 Changed 2 years ago by mkommend

  • Owner changed from mkommend to gkronber
  • Status changed from reviewing to readytorelease

comment:11 Changed 2 years ago by gkronber

r12199: merged r12137, r12151:12152 from trunk to stable

comment:12 Changed 2 years ago by gkronber

  • Resolution set to done
  • Status changed from readytorelease to closed
Note: See TracTickets for help on using tickets.