Ticket #11010 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

IDF outline="yes" breaks geometry cylinder direction

Reported by: Anders Markvardsen Owned by: Harry Jeffery
Priority: critical Milestone: Release 3.4
Component: Framework Keywords:
Cc: m.ganeva@… Blocked By:
Blocking: Tester: Dan Nixon

Description

Load attached IDF which contain one cylinder which should point along the z-axis but is pointing along the x-axis (in instrument view zoom out and the cylinder is alinged along the red line, i.e. the x-axis)

The reason for this is the one use of outline="yes".

To see this in the attached file comment out line 47 and insert line 48 and load this IDF into Mantid

Attachments

DNS_Definition_PAonly_simplified.xml (2.3 KB) - added by Anders Markvardsen 6 years ago.

Change History

Changed 6 years ago by Anders Markvardsen

comment:1 Changed 6 years ago by Harry Jeffery

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Harry Jeffery

  • Status changed from assigned to inprogress

comment:3 Changed 6 years ago by Harry Jeffery

  • Summary changed from IDF outline="yes" brakes geometry cylinder direction to IDF outline="yes" breaks geometry cylinder direction

comment:4 Changed 6 years ago by Harry Jeffery

Refs #11010 Warn if creating outline of insufficient elements

Changeset: 43fe498aaa7714879d1c1d7d83871ba848363f04

comment:5 Changed 6 years ago by Harry Jeffery

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

This is being verified as pull request #261.

comment:6 Changed 6 years ago by Dan Nixon

  • Status changed from verify to verifying
  • Tester set to Dan Nixon

comment:7 Changed 6 years ago by Dan Nixon

Jenkins, retest this please.

comment:8 Changed 6 years ago by Dan Nixon

  • Status changed from verifying to closed

Merge pull request #261 from mantidproject/11010_outline_creation_edge_cases

Issue warning when creating outline with insufficient elements

Full changeset: ed976bbebc0376f847b0d8e7e4667c533991b72e

comment:9 Changed 5 years ago by Nick Draper

Somehow these slipped through without a resolution. Set to Fixed.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11849

Note: See TracTickets for help on using tickets.