Skip to content

Please consider the following formatting changes to #15586 - #43

Closed
alibuild wants to merge 4 commits into
njacazio:nj-alice3-magnetfrom
alibuild:alibot-cleanup-15586
Closed

Please consider the following formatting changes to #15586#43
alibuild wants to merge 4 commits into
njacazio:nj-alice3-magnetfrom
alibuild:alibot-cleanup-15586

Conversation

@alibuild

Copy link
Copy Markdown

Your PR AliceO2Group#15586 cannot be merged as is. You should either run clang-format yourself and update the pull request, or merge this PR in yours.
You can find the AliceO2 coding conventions at https://github.com/AliceO2Group/CodingGuidelines.

njacazio and others added 3 commits August 21, 2026 09:43
Add magnet
Apply clang-format

[MI3] Add superconducting magnet/cryostat geometry to ALICE3 MID simulation

[ALICE3] Magnet: add WindingPack (NbTi+Cu+Al) coil material option

[ALICE3] Magnet: improve WindingPack material documentation

ICN-UNAM: IanMagnet, IanAbsorber, ICNStepped MID layout — 0 overlaps verified

ICN-UNAM: IanAbsorber — Geometria 6 (Antonio): cara externa plana Rmax=290, escalon hacia adentro

ICN-UNAM: cleanup diagnostic instrumentation and translate comments to English

ICN-UNAM: fix hit production — simple sensor names for VMC resolution and reset mHitStarted on Reset()

ICN-UNAM: fix physLay in ProcessHits — extract layer index from sensor name; remove ITS TrackReference boilerplate; guard against invalid physLay

ICN-UNAM: document CurrentVolOffID limitation; full sensor location available via sensor name pattern

ICN-UNAM: IanAbsorber — correct external Rmin to 245 cm (45 cm thickness per Ian reference code)

ICN-UNAM: rename enum values — IanMagnet->SuperconductingMagnet, IanAbsorber->SteppedAbsorber, ICNStepped->SteppedLayout

ICN-UNAM: fix layer 1 half-length to 525 cm (Ian/SD Table 16); revert absorber to Geometry 6 (Rext=290 const, step in Rmin)

ICN-UNAM: fix ALICE3Field.C — B1 in numerator, Rc=170 cm (R_out_coil per Ian; confirmed by A. Ortiz Aug 2026)

Update
Format

UpdateMag
@alibuild
alibuild force-pushed the alibot-cleanup-15586 branch from f9b20bc to 33ea7b9 Compare August 21, 2026 07:45
@njacazio njacazio closed this Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants