Sakib Rahman

Results 8 comments of Sakib Rahman

![CylinderCut](https://user-images.githubusercontent.com/7409132/167418321-b4df3c3a-852c-47cc-9e85-b07696854c9d.JPG) ![Drawing1](https://user-images.githubusercontent.com/7409132/167419227-e7ec48a1-6e84-43ce-9fe9-cf6b417ae144.JPG) A better drawing of the issue constructed in [onshape](https://cad.onshape.com/documents/449cbb673b98f4d6bafd8bd9/w/6a9c4ac0a879643d9b9862ec/e/b81f1828bdd23b7013a3fe06?renderMode=0&uiState=627915c4013ead602bb213b7). In the top diagram, a cylinder with a bevelled shape is constructed and then unwrapped. The circumference of a...

Propose updating the [unit test](https://github.com/rahmans1/acts/blob/bugfix-bevelled-cylinder-surface-and-unit-tests/Tests/UnitTests/Core/Surfaces/CylinderBoundsTests.cpp#L112) in the following manner. Construct the nominal example in onshape. Instead of a single point, grab the coordinates of multiple points along the circumference with...

The boundary condition is implemented as sinusoidal but the unit tests still need to be updated [here](https://github.com/rahmans1/acts/tree/bugfix-bevelled-cylinder-surface-and-unit-tests) .

Suggestions provided by @wdconinc. Just wanted to make sure that nobody else is actively working or planning to work on it. If not, I will be doing it in this...

[Incidental questions](https://indico.bnl.gov/event/14769/contributions/60020/attachments/39718/65871/CutTubeActs.pdf) brought up in the ATHENA-ACTS meeting. 1. **Relevant Issue:** Extending Acts::TGeoSurfaceConverter::cylinderComponents and Acts::TGeoSurfaceConverter::discComponents to support TGeoCtub **Question:** Why is the condition "halfZ>deltaR" used before checking if the shape...

4. **Relevant Issue:** Extending Acts::TGeoSurfaceConverter::cylinderComponents and Acts::TGeoSurfaceConverter::discComponents to support TGeoCtub **Question:** Are tubes with bevel cuts not allowed for discSurfaces? Seems like there is no natural way to extend the...

Intuitively speaking, it doesn't make sense for radial bounds to include a bevel. But we can either force it in for consistency. Or an alternative may be to assign a...

Adding an update to this discussion https://indico.bnl.gov/event/15458/contributions/62426/attachments/40571/67801/ACTS%20integration%20for%20B0%20tracker.pdf