Project

General

Profile

Bug #16015

Dual phase detector geometry dunetpc/gdml/protodunedphase.gdml not correctly loaded

Added by Gianluca Petrillo about 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Start date:
03/28/2017
Due date:
% Done:

100%

Estimated time:
Duration:

Description

While investigating another problem (issue #16007) I attempted to run the geometry test on dunetpc/gdml/protodunedphase.gdml.
The result is a failure. Problems I could observe:

  • computing the weight of the world takes forever (maybe overlapping volumes?)
  • wires are incorrectly sorted on one of the planes (the one associated to view U)

I did not even get as far as to run a real test.
The first step here is to fix DUNE sorter. The point to start from is dunetpc/dune/Geometry/DUNEGeometryHelper_service.cc.

I am attaching the FHiCL file I used to run the wire dump, and the dump itself.
I ran on an area based on dunetpc v06_28_01.

test_geometry_dunedphase.fcl (2.17 KB) test_geometry_dunedphase.fcl wire dump configuration Gianluca Petrillo, 03/28/2017 10:11 PM
geometry_dunedphase_wires.txt.bz2 (65.8 KB) geometry_dunedphase_wires.txt.bz2 wire dump output Gianluca Petrillo, 03/28/2017 10:11 PM

Related issues

Blocks LArSoft - Bug #16007: Wrong charge projection in dual-phase geometryClosed03/28/2017

History

#1 Updated by Gianluca Petrillo about 4 years ago

  • Blocks Bug #16007: Wrong charge projection in dual-phase geometry added

#2 Updated by Gianluca Petrillo about 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Tingjun Yang

In fact, it was me who broke the sorting. The sorting I pushed works for the straight geometry, but not for the rotated one. It was meant for development only, but that was four months ago and obviously I forgot it should not be pushed.

#3 Updated by Gianluca Petrillo about 4 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 0 to 80

Tingjun Tang has pushed a fix in e1cb24441e468204915c9d1ab463654e66dc4ac0 .
The new sorting correctly treats wires that are on the same x / y, y / z or z / x plane.

Geometry dunetpc/gdml/dunedphase10kt_v2.gdml passes the complete geometry test.
Geometry dunetpc/gdml/protodunedphase.gdml still has problems with the test, but the sorting looks correct.

#4 Updated by Tingjun Yang about 4 years ago

Hi Gianluca,

What are the problems with Geometry dunetpc/gdml/protodunedphase.gdml?

Tingjun
Gianluca Petrillo wrote:

Tingjun Tang has pushed a fix in e1cb24441e468204915c9d1ab463654e66dc4ac0 .
The new sorting correctly treats wires that are on the same x / y, y / z or z / x plane.

Geometry dunetpc/gdml/dunedphase10kt_v2.gdml passes the complete geometry test.
Geometry dunetpc/gdml/protodunedphase.gdml still has problems with the test, but the sorting looks correct.

#5 Updated by Gianluca Petrillo almost 4 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 80 to 100

Closing.
There will be some separate check on ProtoDUNE DP geometry.

Also available in: Atom PDF