Skip to content

FT3: build v3b version of the geometry by default#11590

Merged
mconcas merged 4 commits into
AliceO2Group:devfrom
plariono:geom-v3a-bpmod
Jul 10, 2023
Merged

FT3: build v3b version of the geometry by default#11590
mconcas merged 4 commits into
AliceO2Group:devfrom
plariono:geom-v3a-bpmod

Conversation

@plariono
Copy link
Copy Markdown
Contributor

@plariono plariono commented Jun 30, 2023

This PR is to build the FT3 geometry of v3b version from the LOI. Currently it builds the v1.

@plariono plariono requested review from mconcas and qgp as code owners June 30, 2023 13:54
Comment thread Detectors/Upgrades/ALICE3/FT3/simulation/src/Detector.cxx Outdated
@mconcas
Copy link
Copy Markdown
Collaborator

mconcas commented Jun 30, 2023

Thanks @plariono , please find one single comment, you can merge the change from web if you want.

@plariono
Copy link
Copy Markdown
Contributor Author

ciao @mconcas, thank you for the fix. In fact the outer radius of 80 cm corresonds to the version v3b, so I changed also the naming.

@plariono plariono changed the title FT3: build v3a version of the geometry by default FT3: build v3b version of the geometry by default Jun 30, 2023
@mconcas
Copy link
Copy Markdown
Collaborator

mconcas commented Jul 4, 2023

@plariono as soon as you fix the code formatting, we can have CI go through.

Please consider the following formatting changes to AliceO2Group#11590
@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented Jul 4, 2023

Error while checking build/O2/fullCI for afcb677 at 2023-07-04 17:58:

## sw/BUILD/GEANT4_VMC-latest/log
CMake Error at /sw/slc8_x86-64/CMake/v3.23.1-13/share/cmake-3.23/Modules/FindPackageHandleStandardArgs.cmake:230 (message):

Full log here.

@plariono
Copy link
Copy Markdown
Contributor Author

plariono commented Jul 10, 2023

Can this go in please @mconcas ?

@mconcas mconcas merged commit 4ea1ee2 into AliceO2Group:dev Jul 10, 2023
mwinn2 pushed a commit to mwinn2/AliceO2 that referenced this pull request Aug 24, 2023
* Modifications of beampipe

* Changes outer radius, correct version

* Please consider the following formatting changes

---------

Co-authored-by: Pavel Larionov <plariono@alicecerno2.cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants