Ticket #8397 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Extend LoadFullprofResolution to list NPROF

Reported by: Anders Markvardsen Owned by: Karl Palmen
Priority: major Milestone: Release 3.1
Component: Diffraction Keywords:
Cc: zhouw@… Blocked By:
Blocking: Tester: Wenduo Zhou

Description

Extend LoadFullprofResolution to list NPROF in the output table.

Add a row below the top BANK row. I.e. change e.g.

BANK 1 2 3 4 5 6

Alph0 1.59711 1.59711 1.59711 1.59711 1.59711 1.59711

to

BANK 1 2 3 4 5 6

NPROF 13 13 13 13 13 13

Alph0 1.59711 1.59711 1.59711 1.59711 1.59711 1.59711

Change History

comment:1 Changed 7 years ago by Karl Palmen

  • Status changed from new to inprogress

comment:2 Changed 7 years ago by Karl Palmen

Add NPROF row to OutputTableWorkspace re #8397

row still to be populated with correct NPROF number

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 9d84c52f0abd49918477fb68812c9f6d550b7ff1

comment:3 Changed 7 years ago by Karl Palmen

Get NPROF number into OutputTableWorkspace re #8397

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 4ef85324cd038c9edfc2d4c882762034247beb86

comment:4 Changed 7 years ago by Karl Palmen

Add unit test for NPROF re #8397

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 4a3e15feb3426f36559aa3c7cf77def0e619448e

comment:5 Changed 7 years ago by Karl Palmen

Remove warning about not processing NROF line re #8397

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: ebaf07a6d1df9a5acff654f6f7fa351a9f7272c0

comment:6 Changed 7 years ago by Karl Palmen

Use lexical_cast to get integer from string re #8397

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 507ab4c0ff326189586173838866831d9208240c

comment:7 Changed 7 years ago by Karl Palmen

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

comment:8 Changed 7 years ago by Wenduo Zhou

  • Status changed from verify to verifying
  • Tester set to Wenduo Zhou

comment:9 Changed 7 years ago by Wenduo Zhou

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8397_nprof_number'

Full changeset: 694f62cd5ba0983bf2db6899ba6fefaeaa09d64c

Test is passed as NPROF is added to output table workspace as a new row. The only difference from the description of this ticket is that NPROF is not the second row just below 'BANK'. It shouldn't affect how to use this table workspace. The table rows are ordered by alphabet of parameter name except 'BANK' as it was implemented.

Last edited 7 years ago by Wenduo Zhou (previous) (diff)

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9242

Note: See TracTickets for help on using tickets.