Sakib Rahman
Sakib Rahman
  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