Chandradoy Chatterjee
Chandradoy Chatterjee
Please don't merge now!
> LGTM. Mark ready and add to merge queue. @c-dilks I had a discussion with Macro today, and he is right. That this gap should not be carbon fibre but...
@c-dilks I checked the conflicts; there are two conflicts wrt main. The first one related to the mirror description and the second one is related to the sensor X,Y,Z and...
Here are the ring images with the current configuration; dual mirror:  single mirror: 
The photons on the sensor surface: 
The photons hits not very clean (5 tracks in 24 phi bins and 3 eta bins) dual mirror:  Chris' PR:  To be addressed
## The validation method is adopted as the following - To use single mirror configuration each time with one of the two mirrors' parameters. - Extract the resolution using single...
I will implement it.
Are you sure these 20 and 35.2 are not the diameters but radii?
I have wrote Marco for a final confirmation. Anyway, the working branch is `dRICH_envelope_07052023`