Ticket #1712 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

<component-link> apply to all components with a given name

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: major Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

Michael found that when using e.g.

<component-link name="pixel">

<parameter name="tube_pressure">

<value units="atm" val="10.0"/>

</parameter>

</component>

in an IDF then the parameter "tube_pressure" was only applied to the first component encountered with the name "pixel".

Change this so that apply to all components

Change History

comment:1 Changed 10 years ago by Anders Markvardsen

(In [6660]) Code for: <component-link> apply to all components with a given name. Wiki updated Added tests In DUM_Defintion.xml changed component_link with component-link.

re #1712

comment:2 Changed 10 years ago by Anders Markvardsen

  • Status changed from new to accepted

comment:3 Changed 10 years ago by Anders Markvardsen

(In [6716]) Added a few more tests using getNumberParameter instead of get to get hold of parameter values in connection with <component-link> fix.

re #1712

comment:4 Changed 10 years ago by Russell Taylor

(In [6726]) Disable failing test pending Anders' attention. Re #1712.

comment:5 Changed 10 years ago by Anders Markvardsen

(In [6733]) Updated tests for <component-link> fix.

re #1712

comment:6 Changed 10 years ago by Anders Markvardsen

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

comment:7 Changed 10 years ago by Anders Markvardsen

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:8 Changed 10 years ago by Anders Markvardsen

(In [6750]) Few more tests to demonstrate recursive look-up of parameters. re #1712

comment:9 Changed 10 years ago by Anders Markvardsen

  • Status changed from reopened to accepted

Added info about accessing parameters to IDF wiki description

comment:10 Changed 10 years ago by Anders Markvardsen

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

comment:11 Changed 10 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:12 Changed 10 years ago by Michael Reuter

  • Status changed from verifying to closed

I have verified this using the DUM_Definition.xml and applying the case given in the description. The parameters are correctly seen by each pixel object.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2559

Note: See TracTickets for help on using tickets.