Ticket #9987 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Indirect Reduction multi run naming convention

Reported by: Samuel Jackson Owned by: Karl Palmen
Priority: major Milestone: Release 3.3
Component: Indirect Inelastic Keywords:
Cc: spencer.howells@… Blocked By:
Blocking: Tester: Alex Buts

Description (last modified by Samuel Jackson) (diff)

During an energy reduction, using a range of files and summing them forces the result to take on the name of one of the files used in the range.

This make it not immediately obvious that the run is a range of files. The solution discussed with Sanghamitra and Spencer is to:

  • Add an indicator to the name to indicate multiple runs
    • e.g. irs00001_multi_graphite002_red
  • Add a sample log listing the runs numbers used in this file.

Change History

comment:1 Changed 6 years ago by Samuel Jackson

  • Summary changed from Multi run naming convention to Indirect Reduction multi run naming convention

comment:2 Changed 6 years ago by Samuel Jackson

  • Description modified (diff)

comment:3 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:4 Changed 6 years ago by Dan Nixon

  • Owner changed from Samuel Jackson to Dan Nixon
  • Blocked By 10083 added

This is something I will add alongside #10083

comment:5 Changed 6 years ago by Dan Nixon

  • Cc spencer.howells@… added

comment:6 Changed 6 years ago by Dan Nixon

  • Blocked By 10083 removed

This has been added in #10083

comment:7 Changed 6 years ago by Dan Nixon

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

comment:8 Changed 6 years ago by Karl Palmen

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

comment:9 Changed 6 years ago by Karl Palmen

  • Status changed from verifying to reopened
  • Resolution invalid deleted

I have not found the change in #10083, where this has occurred. Please give the comment number of this change in #10083.

comment:10 Changed 6 years ago by Dan Nixon

Comments 15, 16 and 20 in #10083 are for the commits that correct this.

comment:11 Changed 6 years ago by Karl Palmen

  • Status changed from reopened to verify
  • Owner changed from Dan Nixon to Karl Palmen
  • Resolution set to fixed

comment:12 Changed 6 years ago by Harry Jeffery

  • Status changed from verify to verifying
  • Tester changed from Karl Palmen to Harry Jeffery

comment:13 Changed 6 years ago by Harry Jeffery

  • Status changed from verifying to verify
  • Tester Harry Jeffery deleted

I was unable to load a data set that I could test with. Returning to testing pool.

comment:14 Changed 6 years ago by Alex Buts

  • Status changed from verify to verifying
  • Tester set to Alex Buts

comment:15 Changed 6 years ago by Alex Buts

Not easy to find the log name as a run_number log exists and shows first run number when the contributing run numbers are listed in multi_run_numbers log entry.I would rather modify run_number entry but this is for customer to decide.

Fixed otherwise.

comment:16 Changed 6 years ago by Alex Buts

  • Status changed from verifying to closed

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10829

Note: See TracTickets for help on using tickets.