Skip to content

Update TV XML to Control Sensor/Pixel Size only + Remove Reliance on TVConstants

Dan Thompsonrequested to merge
dathomps_TVxmlSimplify into master

Remove the pixel layout controls, and use a silicon X,Y size control which sets the N pixels dependent on the PixelSize set.

Idea is to set "PicoPix" sensors with:

<!-- Pixel size -->
<constant name = "TV:PixelSize"         value = "0.05*mm"/>

<constant name = "TV:ChipSizeX"         value = "12.8*mm"/>
<constant name = "TV:ChipSizeY"         value = "12.8*mm"/>

and "IGNITE" with:

<!-- Pixel size -->
<constant name = "TV:PixelSize"         value = "0.045*mm"/>

<constant name = "TV:ChipSizeX"         value = "14.4*mm"/>
<constant name = "TV:ChipSizeY"         value = "14.4*mm"/>

ToDo:

  • Clean-up of xml comments and old junk when ready to merge
  • Add changes to replace TVConstants with info from XML (add new members to DeTV and DeTVSensor)
  • Search for use of static numbers (2 modules per station, 4 sensors per module, etc., and replace with members
  • Prepare corresponding MRs on Rec / LHCb (just Rec?) (again searching for static number usage)
  • Ensure TV Boole MRs pick this change up

cc: @tevans

Validated by

  • Core Software
  • RTA
  • Simulation
Edited by Dan Thompson

Merge request reports