Fix MUON station dimensions (!300) · Merge requests · LHCb / Detector · GitLab

Snippets Groups Projects

Merged Flavio Archilli requested to merge MuonDev_GeoInfo into master 2 years ago

Issue with inner and outer muon detector dimensions (reported below) is solved Rec#407 (comment 6066825)

Edited 2 years ago by Flavio Archilli

Activity

Flavio Archilli assigned to @clemenci, @jonrob, and @desimone 2 years ago

assigned to @clemenci, @jonrob, and @desimone
Flavio Archilli changed the description 2 years ago

changed the description
Christopher Rob Jones added Muon lhcb-dd4hep labels 2 years ago

added Muon lhcb-dd4hep labels
Christopher Rob Jones mentioned in merge request Rec!3142 (merged) 2 years ago

mentioned in merge request Rec!3142 (merged)
Christopher Rob Jones @jonrob · 2 years ago

Maintainer

Resolved 2 years ago by Roel Aaij

FYI @raaij

Will this also (help to) address #36 ?

Last reply by Roel Aaij 2 years ago
Flavio Archilli added 1 commit 2 years ago
added 1 commit

bc41fa03 - clean up

Compare with previous version
Christopher Rob Jones changed title from Draft: Muon station dimensions fixed to Draft: Fix MUON station dimensions 2 years ago

changed title from Draft: Muon station dimensions fixed to Draft: Fix MUON station dimensions
Roel Aaij @raaij started a thread on an old version of the diff 2 years ago

Resolved 2 years ago by Flavio Archilli
Last reply by Flavio Archilli 2 years ago

Software for LHCb @lhcbsoft · 2 years ago

Developer

[2022-10-12 00:03] Validation started with lhcb-dd4hep-rootunits#64
[2022-10-12 00:03] Validation started with lhcb-dd4hep#1294
[2022-10-13 00:07] Validation started with lhcb-dd4hep-rootunits#65
[2022-10-13 00:07] Validation started with lhcb-dd4hep#1295
[2022-10-14 00:03] Validation started with lhcb-dd4hep-rootunits#66
[2022-10-14 00:03] Validation started with lhcb-dd4hep#1296
[2022-10-15 00:11] Validation started with lhcb-dd4hep-rootunits#67
[2022-10-15 00:12] Validation started with lhcb-dd4hep#1297

Edited 2 years ago by Software for LHCb

Flavio Archilli added 1 commit 2 years ago

added 1 commit

86532518 - solving issue\!36 (merged)

Compare with previous version

Flavio Archilli added 1 commit 2 years ago

added 1 commit

97063018 - fix formatting

Compare with previous version

Flavio Archilli @farchill · 2 years ago

Author Developer

Hi @raaij, this last commit should fix the issue mentioned here #36

Flavio Archilli added 1 commit 2 years ago

added 1 commit

0dbd4e64 - tile position and size fixed

Compare with previous version

Flavio Archilli @farchill · 2 years ago

Author Developer

Resolved 2 years ago by Christopher Rob Jones

Morning @raaij, @gligorov , @bsciasci , the last commit fixes X,Y, (), DX,DY,DZ as we thought #36 (comment 6090976). The Z position is still 100mm away but @desimone @bcouturi and @tlatham are working on it. @raaij can test it?

Edited 2 years ago by Flavio Archilli
Last reply by Patrizia De Simone 2 years ago

Flavio Archilli mentioned in issue #36 2 years ago

mentioned in issue #36

Roel Aaij mentioned in merge request Allen!853 (closed) 2 years ago

mentioned in merge request Allen!853 (closed)

Patrizia De Simone added 1 commit 2 years ago

added 1 commit

46e3a359 - align M#(A/C)SideYOffset numbers in MuonParameters.xml and get BeamAngle from global/parameters.xml

Compare with previous version

Patrizia De Simone added 1 commit 2 years ago

added 1 commit

f481abfb - changes in test_demuon

Compare with previous version

Patrizia De Simone added 1 commit 2 years ago

added 1 commit

744a954d - Fixed formatting

Compare with previous version

Flavio Archilli added 1 commit 2 years ago

added 1 commit

0c104d8b - code improved thanks to Marco Clemencic

Compare with previous version

Please register or sign in to reply