Chandradoy Chatterjee

Results 25 comments of 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: ![Screenshot from 2023-04-18 23-22-10](https://user-images.githubusercontent.com/52901208/233018458-8e285b5f-eada-4ec8-85aa-5141adf2e883.png) single mirror: ![Screenshot from 2023-04-19 11-16-12](https://user-images.githubusercontent.com/52901208/233028898-70eb103b-d829-4896-b279-cf16fa707391.png)

The photons on the sensor surface: ![Screenshot from 2023-04-19 11-13-27](https://user-images.githubusercontent.com/52901208/233028204-143d795f-bbc5-4bde-8f66-0627448376aa.png)

The photons hits not very clean (5 tracks in 24 phi bins and 3 eta bins) dual mirror: ![image](https://user-images.githubusercontent.com/52901208/233083366-5e7b184d-fdd0-4570-9863-06e401d78409.png) Chris' PR: ![image](https://user-images.githubusercontent.com/52901208/233083522-9925e670-4143-48de-96e0-bccd2f2eef01.png) 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...

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`