Ticket #10669 (closed: wontfix)

Opened 6 years ago

Last modified 5 years ago

Plotting counts/bin width What should the default be?

Reported by: Nick Draper Owned by: Nick Draper
Priority: critical Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By: #10639
Blocking: Tester: Andrei Savici

Description

re #10639.

We need to make sure the default is set "correctly" for this release

Talk to some instrument scientists and come to a conclusion.

Change History

comment:1 Changed 6 years ago by Nick Draper

  • Blocked By 10639 added

(In #10639) Reopened due to a few issues: only the first should be dealt with in this ticket, the others have new tickets.

  1. If you plot errors, whether by plotspectrum with errors, or the graph->add error bars -> from workspace, they are not scaled by the bin width and therefore overpower the data.
  2. The interaction with fitting is confusing as you setup the plot against normalized data, but it fits against the underlying non normalized data. - Follow on ticket to deal with this #10668.
  3. We need to make sure the default is set "correctly" for this release - Follow on ticket to determine this and implement #10669.

comment:2 Changed 6 years ago by Nick Draper

In a quick poll of instrument scientsts at ISIS the following scientists voted as follows:

default to normalize by bin width

  • Stephen King
  • Tim Charlton
  • Toby Perring
  • Helen Walker
  • Rob Dlagleish
  • Ann Terry
  • Alan Soper

Default to remain as it is

  • Sanghamitra M (Indirect Inelastic)

Sanghamitra's concern were primarily about what it might do to graph further down the reduction chain, and the change from the norm, rather than a specific concern about a specific problem it would cause.

Still need to get feedback from the Muon group

comment:3 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:4 Changed 6 years ago by Nick Draper

Update from Pascal:

we should normalize to bin width

Stephen Cotrell (Muons):

They would not want this to affect assymetry plots - Anders will check if these are point data.

comment:5 Changed 6 years ago by Nick Draper

  • Status changed from assigned to verify
  • Resolution set to wontfix

The consensus is to go ahead with this on by default, at least into beta testing.

No Code Changes to test.

comment:6 Changed 6 years ago by Andrei Savici

  • Status changed from verify to verifying
  • Tester set to Andrei Savici

comment:7 Changed 6 years ago by Andrei Savici

  • Status changed from verifying to closed

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11511

Note: See TracTickets for help on using tickets.