Ticket #5919 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Not all points are plotted when using IDA

Reported by: Dereck Kachere Owned by: Dereck Kachere
Priority: major Milestone: Release 2.3
Component: Mantid Keywords: PatchCandidate,Released
Cc: Blocked By:
Blocking: Tester: Roman Tolchenov

Description

When trying to plot results using the 'Run' button in Indirect Data Analysis when 'Plot Result' is checked, not all points appear - only part of the graph is drawn. However, the graph is plotted just fine when you draw it by right-clicking in the 'Workspaces' area and selecting 'Plot spectrum...'

Attachments

indirect_analysis_bug.png (293.5 KB) - added by Dereck Kachere 8 years ago.
indirect_analysis_bug_2.png (328.1 KB) - added by Dereck Kachere 8 years ago.

Change History

Changed 8 years ago by Dereck Kachere

Changed 8 years ago by Dereck Kachere

comment:1 Changed 8 years ago by Dereck Kachere

  • Keywords PatchCandidate added

comment:2 Changed 8 years ago by Dereck Kachere

Refs #5919. Edited code to allow drawing all points

All points are now drawn. The rescaling of the x-axis is not quite the same as if you right-click and 'Plot Spectrum...' though.

Changeset: f9ce051b3a7a58326a65ff9a0f76797f2d5d0153

comment:3 Changed 8 years ago by Dereck Kachere

  • Status changed from new to accepted

comment:4 Changed 8 years ago by Dereck Kachere

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

comment:5 Changed 8 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:6 Changed 8 years ago by Roman Tolchenov

  • Status changed from verifying to reopened
  • Resolution fixed deleted

The old code in IndirectDataAnalysis.py somehow came back (lines 179-189) and reverted the fix. Also removing the directory part from the file name at line 148 could make LoadNexus fail and mantid to crash.

comment:7 Changed 8 years ago by Dereck Kachere

  • Status changed from reopened to accepted

comment:8 Changed 8 years ago by Dereck Kachere

refs #5919 Reverted changes made by adding a file not fully up to date

Code which Spencer updated did not have the change I had originally made.

refs #6023 as well. Changed code so that when LoadNexus is run, the file's absolute path does not need to be entered.

Changeset: d0598be25c3ad0da19a58048189f9a909b90c5c5

comment:9 Changed 8 years ago by Dereck Kachere

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

comment:10 Changed 8 years ago by Roman Tolchenov

  • Status changed from verify to verifying

comment:11 Changed 8 years ago by Roman Tolchenov

  • Status changed from verifying to reopened
  • Resolution fixed deleted

LoadNexus has to take the full path to the file.

comment:12 Changed 8 years ago by Dereck Kachere

  • Status changed from reopened to accepted

comment:13 Changed 8 years ago by Dereck Kachere

refs #5919 Change for LoadNexus to take full path to file

Changeset: 2e3cdc8ab9e5c98b51cbbb27b852573c4d19841e

comment:14 Changed 8 years ago by Dereck Kachere

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

comment:15 Changed 8 years ago by Roman Tolchenov

  • Status changed from verify to verifying

comment:16 Changed 8 years ago by Roman Tolchenov

  • Status changed from verifying to closed

comment:17 Changed 8 years ago by Dereck Kachere

Refs #5919. Edited code to allow drawing all points

All points are now drawn. The rescaling of the x-axis is not quite the same as if you right-click and 'Plot Spectrum...' though.

Changeset: f9ce051b3a7a58326a65ff9a0f76797f2d5d0153

comment:18 Changed 8 years ago by Dereck Kachere

refs #5919 Reverted changes made by adding a file not fully up to date

Code which Spencer updated did not have the change I had originally made.

refs #6023 as well. Changed code so that when LoadNexus is run, the file's absolute path does not need to be entered.

Changeset: d0598be25c3ad0da19a58048189f9a909b90c5c5

comment:19 Changed 8 years ago by Dereck Kachere

refs #5919 Change for LoadNexus to take full path to file

Changeset: 2e3cdc8ab9e5c98b51cbbb27b852573c4d19841e

comment:20 Changed 8 years ago by Dereck Kachere

refs #5919 Reverted changes made by adding a file not fully up to date

Code which Spencer updated did not have the change I had originally made.

refs #6023 as well. Changed code so that when LoadNexus is run, the file's absolute path does not need to be entered.

Changeset: d0598be25c3ad0da19a58048189f9a909b90c5c5

comment:21 Changed 8 years ago by Dereck Kachere

refs #5919 Change for LoadNexus to take full path to file

Changeset: 2e3cdc8ab9e5c98b51cbbb27b852573c4d19841e

comment:22 Changed 7 years ago by Nick Draper

  • Keywords PatchCandidate,Released added; PatchCandidate removed

comment:23 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6765

Note: See TracTickets for help on using tickets.