Ticket #7711 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

[IDA] Elwin does strange things with temperature.

Reported by: Samuel Jackson Owned by: Samuel Jackson
Priority: major Milestone: Release 3.0
Component: Indirect Inelastic Keywords:
Cc: Blocked By: #7735
Blocking: Tester: Gesner Passos

Description

Elwin is producing strange output under some circumstances during the generation of the _elf files.

Sometimes, plotting the _elf file causes the x axis to not be set to the correct scale, causing nothing to appear in the plot. Also, the labels on the axis often not correct. It should read "Temperature" but is instead showing either "Quantity" or "Run number".

Doing a fresh generation of the files using C2E and then running them through Elwin produces regular output, but saving them and then reloading them appears to cause issues.

It looks like Elwin may be falling back on some default values under some circumstances.

Attachments

irs55128_graphite002_red.nxs (184.6 KB) - added by Samuel Jackson 7 years ago.
irs55129_graphite002_red.nxs (228.7 KB) - added by Samuel Jackson 7 years ago.
irs55130_graphite002_red.nxs (228.8 KB) - added by Samuel Jackson 7 years ago.
irs55131_graphite002_red.nxs (228.9 KB) - added by Samuel Jackson 7 years ago.

Change History

comment:1 Changed 7 years ago by Samuel Jackson

  • Owner set to Samuel Jackson

comment:2 Changed 7 years ago by Samuel Jackson

  • type changed from enhancement to defect

comment:3 Changed 7 years ago by Samuel Jackson

The issue with "Quantity" is a separate, wider issue to do with file saving/loading and will be dealt with in ticket #7735

comment:4 Changed 7 years ago by Samuel Jackson

  • Blocked By 7735 added

comment:5 Changed 7 years ago by Samuel Jackson

  • Status changed from new to inprogress

Adding add sample log to Elwin

Refs #7711

Changeset: 7122aa6b761107d9c8e66c5a76e80070c04c92d6

comment:6 Changed 7 years ago by Samuel Jackson

Refs #7711 Added check for whether temp is decreasing.

Changeset: d1fcb4d339bdd4540e1f000710045a6d8bcc7d0d

comment:7 Changed 7 years ago by Samuel Jackson

Merge branch 'bugfix/7711_elwin_temperature_fix' into develop

Refs #7711

Conflicts:

Code/Mantid/scripts/Inelastic/IndirectDataAnalysis.py

Changeset: 315ac507ccdfaae2fb00eaef9e87f09c71641809

Changed 7 years ago by Samuel Jackson

Changed 7 years ago by Samuel Jackson

Changed 7 years ago by Samuel Jackson

Changed 7 years ago by Samuel Jackson

comment:8 Changed 7 years ago by Samuel Jackson

  • Status changed from inprogress to verify
  • Component changed from Framework to Indirect Inelastic
  • Resolution set to fixed

To Tester

The problem with the x axis not plotting has been resolved in this ticket. The issue was down to the temperature (plotted on the x) decreasing in some runs meaning that plot spectrum became confused when the stating value was greater than the end value.

Use the supplied files as input to Elwin and plot the generated *_elf workspace. The graph should plot with a reasonable scale and temperature should descend towards the right.

Last edited 7 years ago by Samuel Jackson (previous) (diff)

comment:9 Changed 7 years ago by Samuel Jackson

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:10 Changed 7 years ago by Samuel Jackson

  • Status changed from reopened to inprogress

Refs #7711 Forgot to flip Y and E axes as well.

Changeset: 2bd12407dc727b0720f31039720aa325a1c3c2d1

comment:11 Changed 7 years ago by Samuel Jackson

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

comment:12 Changed 7 years ago by Gesner Passos

  • Status changed from verify to verifying
  • Tester set to Gesner Passos

comment:13 Changed 7 years ago by Gesner Passos

  • Status changed from verifying to reopened
  • Resolution fixed deleted

Everything works fine, but we have agreed that the notation:

datTx = datTx[::-1]

could be replaced by:

datTx.reverse()

improving readability.

comment:14 Changed 7 years ago by Samuel Jackson

  • Status changed from reopened to inprogress

Refs #7711 Add comment to obscure code.

Changeset: 7249c7371acae5921307b689c92fe3e845ed8e5a

comment:15 Changed 7 years ago by Samuel Jackson

Refs #7711 Merge Master into branch to resolve conflict.

Changeset: f16ed8d9506dfdd2941d0d36d47c8f1276ee33b5

comment:16 Changed 7 years ago by Samuel Jackson

As discussed these are numpy arrays which don't have the reverse method. Instead I've added a comment to make it clearer what is happening.

comment:17 Changed 7 years ago by Samuel Jackson

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

comment:18 Changed 7 years ago by Gesner Passos

  • Status changed from verify to verifying

comment:19 Changed 7 years ago by Gesner Passos

It was my fault, there is not reverse for the numpy array data type, and the way to reverse is through that code. So, now, a comment was given to add understanding.

comment:20 Changed 7 years ago by Samuel Jackson

  • Status changed from verifying to closed

Merge branch 'bugfix/7711_elwin_temperature_fix' into feature/7711_elwin_temp_fix_2

Full changeset: a00ff657240a02a71afa6e17114e5d639809bbd3

comment:21 Changed 7 years ago by Gesner Passos

Merge remote-tracking branch 'origin/feature/7711_elwin_temp_fix_2'

Full changeset: 8cf1052aafa6ce069203deafa48426937a92547d

comment:22 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 8556

Note: See TracTickets for help on using tickets.