Ticket #8070 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Remove all zeros from ascii file output and add switch that will allow two different way to output overlap data

Reported by: Jean Bilheux Owned by: Jean Bilheux
Priority: major Milestone: Release 3.0
Component: Reflectometry Keywords:
Cc: Blocked By:
Blocking: Tester: Mathieu Doucet

Description

The switch, off by default, will, in case there are two data points with the same x value, keep the one with the lowest error bar. Otherwise, it will calculate the average of the two points (like it does right now).

Change History

comment:1 Changed 7 years ago by Jean Bilheux

  • Status changed from new to inprogress
  • Owner set to Jean Bilheux

comment:2 Changed 7 years ago by JeanBilheux

Implemented algorithm that remove the 0 values. This refs #8070

Changeset: 8e6f971966941e4fc2d9514d4c3406452d1caf1a

comment:3 Changed 7 years ago by Jean Bilheux

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:4 Changed 7 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:5 Changed 7 years ago by Karl Palmen

  • Status changed from verifying to verify
  • Tester Karl Palmen deleted

I thought it was SaveAscii, but it was something else. Test needs explaining.

comment:6 Changed 7 years ago by Mathieu Doucet

  • Status changed from verify to verifying
  • Tester set to Mathieu Doucet

comment:7 Changed 7 years ago by Mathieu Doucet

Passing this ticket since it solves the issue. Will need to check with the instrument scientist whether the lower limit of 1e-15 is appropriate in the long term.

comment:8 Changed 7 years ago by Mathieu Doucet

  • Status changed from verifying to closed

Merge remote branch 'origin/feature/8070_new_output_ascii_format_flag'

Full changeset: 5b7aba2b812a3fa8578af58da5edd952296ecdeb

comment:9 Changed 7 years ago by Nick Draper

  • Component changed from Framework to Reflectometry

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8915

Note: See TracTickets for help on using tickets.