Ticket #179 (closed Enhancement: Fixed)

Opened 3 years ago

Last modified 19 months ago

All mean and std.dev values should be calculated in both linear and dB

Reported by: cp Owned by: jean-philippe
Priority: Minor Milestone: 3.2.3 - Beta 1 - P1546
Component: GUI Version:
Keywords: Cc:
Blocked By: Blocking:
Estimated Number of Hours: 0 Pending: no
Add Hours to Ticket: 0 Total Hours: 0
Original Estimate: 0 Source (can be left blank):
Must be closed before next release: no Effort Required: Unknown

Description

Today SEAMCAT calculates all averages as linear average of dBm values.

Attachments

figure1.gif Download (21.1 KB) - added by jean-philippe 20 months ago.
figure2.gif Download (21.3 KB) - added by jean-philippe 20 months ago.

Change History

comment:1 Changed 2 years ago by jean-philippe

  • Effort Required set to Unknown
  • Milestone changed from 3.2.0 to 3.2.0 Beta 19

there is a need for further specification on this. For instance, on the display of the results. Should we show all the results or do we have a switch to select what is to be shown?

need STG feedback.

comment:2 Changed 2 years ago by jean-philippe

  • Milestone changed from 3.2.0 Beta 19 to Need feedback from STG

comment:3 Changed 2 years ago by jean-philippe

These values are presented only for display just for information. No decision in STG21 21 on this tickets. This need further consideration.

comment:4 Changed 20 months ago by jean-philippe

STG23 decided to add 2 extra columns in the simulation sumary:

  • Mean and stdDev derived from linear values (bold)
  • Mean and stdDev derived from dBm values (on the right hand side)

And add a note in the handbook explaining why the dBm calculation is kept (i.e. to enable backward version results comparison)

Target: 3.2.3 release (to check with developer how long it take)

Last edited 20 months ago by jean-philippe (previous) (diff)

Changed 20 months ago by jean-philippe

Changed 20 months ago by jean-philippe

comment:5 Changed 20 months ago by jean-philippe

  • Milestone changed from Need feedback from STG to 3.2.3 - Beta 1 - P1546

comment:6 Changed 19 months ago by thomas.thorndahl

Implemented for the simulation summary in rev. [845]

comment:7 Changed 19 months ago by thomas.thorndahl

Text bold and swapped dBm and linear panels [846]

comment:8 Changed 19 months ago by thomas.thorndahl

Correct texts in bold [847]

comment:9 Changed 19 months ago by thomas.thorndahl

Moved into one panel again [849]

comment:10 Changed 19 months ago by thomas.thorndahl

dRSS linear and dRSS dBm was swapped [850]

comment:11 Changed 19 months ago by jean-philippe

[860] was also a fixe for this ticket. The description of the commit is for another ticket.

Last edited 19 months ago by jean-philippe (previous) (diff)

comment:12 Changed 19 months ago by jean-philippe

[851] is also a fix to this ticket.

comment:13 Changed 19 months ago by thomas.thorndahl

Partly reverted to original display of linear mean and stddev in [861] as agreed.

comment:14 Changed 19 months ago by jean-philippe

during the implementation, it was found out that the mean value derived from linear calculation exhibit high fluctuation, while the mean value derived from dBm remain constant.

The calculation are correct but displaying such information as originally agreed by STG is not very usefull.

After discussion with Karl loew, it is proposed to replace the mean (linear) in the display by the median (linear). The difference is negligible between the median derived from the dBm or linear values. These values are also very close to the mean derived from the dBm values.

STG was requested to provide feedback. no response received. silence is approval.

the median(linear) will be proposed for revision in the next beta release.

comment:15 Changed 19 months ago by thomas.thorndahl

Median display implemented in [861] and [862]

comment:16 Changed 19 months ago by jean-philippe

fix made in [863]

comment:17 Changed 19 months ago by thomas.thorndahl

Implemented median display for drss/irss-signaldisplays [864]

comment:18 Changed 19 months ago by jean-philippe

  • Status changed from new to closed
  • Resolution set to Fixed

tested with matlab. ticket is closed.

Note: See TracTickets for help on using tickets.