GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — ALICE3 Forward Detector update by pkurash · Pull Request #15791 · AliceO2Group/AliceO2

Conversation

@pkurash

@pkurash pkurash commented Sep 12, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Updated code for ALICE3 Forward Detector (FD).

  1. Scintillator modules placed closer to the interaction point
  2. Modified eta coverage
  3. Two extra Cherenkov radiator arrays

For more details, see my presentation at 7th Upgrade Week:

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
pkurash added 30 commits July 15, 2026 11:57
@pkurash
new geometry
680cdb6
@pkurash
new geometry
de62c6c
@pkurash
added asymmetric configuration
986b4c1
@pkurash
added asymmetric configuration
ce48a1c
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
eb918fb
@pkurash
Added functions for Cherenkov module segmentation
e51e253
@pkurash
Hits fixed
71ee836
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
93d3b0e
@pkurash
Hits fixed
184aadc
@pkurash
Three differen functions for building Cherenkov modules
c818493
@pkurash
introduced new constants
f877601
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
88209c7
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
6216bbb
@pkurash
fix cherenkov geometry
eaadb0c
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
f751955
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
52b61fd
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
a159007
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
ea42210
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
f76da28
@pkurash
remove unnecessary logs
3e7d675
@pkurash
Changed Cherenkov modules numbering
19389d4
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
c9f9c9e
@pkurash
Cherenkov channels numbering
e4a37eb
@pkurash
Merge branch 'new_geometry' of github.com:pkurash/AliceO2 into new_ge…
9748ff9
…ometry
@pkurash
Set sizes of quartz radiators
23d6e28
@pkurash
corrected radiator module distance
8aa625f
@pkurash
corrected radiator module distance
b2c77b2
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
b8f838e
@pkurash
corrected scintillator module position
0a7a750
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
ffea2f7
njacazio
njacazio previously approved these changes Sep 23, 2026
View reviewed changes
@njacazio
njacazio enabled auto-merge (squash) September 23, 2026 19:57
@njacazio
njacazio disabled auto-merge September 23, 2026 19:57
@njacazio
njacazio enabled auto-merge (squash) September 23, 2026 19:58
@pkurash
updated radiator sizes
95d2d94
auto-merge was automatically disabled September 25, 2026 06:29

Head branch was pushed to by a user without write access

@pkurash
pkurash dismissed njacazio’s stale review via 95d2d94 September 25, 2026 06:29
@alibuild

alibuild commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 95d2d94 at 2026-09-25 08:46:

No log files found

Full log here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
d4604fa
@njacazio
njacazio enabled auto-merge (squash) September 27, 2026 14:19
njacazio
njacazio approved these changes Sep 27, 2026
View reviewed changes
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
58dab5d
@alibuild

alibuild commented Oct 6, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 58dab5d at 2026-10-07 05:18:

## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -a -v clang-diagnostic-error error-log.txt
++ grep -a ' error:'
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/FD3/simulation/src/Detector.cxx:383:12: error: statement should be inside braces [readability-braces-around-statements]
++ [[ 0 == 0 ]]
++ exit 1
--

Full log here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
6c80154
@alibuild

alibuild commented Oct 7, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 6c80154 at 2026-10-12 03:51:

## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -a -v clang-diagnostic-error error-log.txt
++ grep -a ' error:'
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/FD3/simulation/src/Detector.cxx:383:12: error: statement should be inside braces [readability-braces-around-statements]
++ [[ 0 == 0 ]]
++ exit 1
--

Full log here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

njacazio
njacazio approved these changes Oct 8, 2026
View reviewed changes
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
7bb02fa
njacazio
njacazio previously approved these changes Oct 13, 2026
View reviewed changes
@alibuild

alibuild commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for 7bb02fa at 2026-10-13 14:27:

## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -a -v clang-diagnostic-error error-log.txt
++ grep -a ' error:'
/sw/SOURCES/O2/slc9_x86-64-slc9_x86-64/0/Detectors/Upgrades/ALICE3/FD3/simulation/src/Detector.cxx:383:12: error: statement should be inside braces [readability-braces-around-statements]
++ [[ 0 == 0 ]]
++ exit 1
--

Full log here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pkurash
small changes
52c0ea3
auto-merge was automatically disabled October 13, 2026 13:05

Head branch was pushed to by a user without write access

@pkurash
pkurash dismissed njacazio’s stale review via 52c0ea3 October 13, 2026 13:05
@njacazio

njacazio commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

Hi @pkurash many thanks! Would it be possible to rebase to the dev branch? I have the feeling that we changed things already. You can try this with git pull --rebase upstream dev or similar? You can let me know if you need any help! Many thanks!!

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pkurash

pkurash commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

Hi @pkurash many thanks! Would it be possible to rebase to the dev branch? I have the feeling that we changed things already. You can try this with git pull --rebase upstream dev or similar? You can let me know if you need any help! Many thanks!!

Hello, I have tried this command, the pull created conflicts which I resolved manually with mergetool and removed *orig files, now I have "HEAD detached from refs/heads/new_geometry"

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
740e7a0
@njacazio njacazio mentioned this pull request Oct 15, 2026
ALICE3 Forward Detector update - #15791 #15967
Open
@pkurash
Merge branch 'AliceO2Group:dev' into new_geometry
a8462e9
@alibuild

alibuild commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2/fullCI_slc9 for a8462e9 at 2026-10-22 10:59:

## sw/BUILD/O2-sim-challenge-test-latest/log
./sim-challenge.log[ERROR] Error in fetching object http://127.0.0.1:36501/ccdb-prod/TPC/Calib/PadGainFull/1546300800000/, curl response code:500
./sim-challenge.log[FATAL] Got nullptr from CCDB for path TPC/Calib/PadGainFull and timestamp 1546300800000
./sim-challenge.log[FATAL] unknown error while setting up workflow in o2-sim-digitizer-workflow.
./digi.log[ERROR] Error in fetching object http://127.0.0.1:36501/ccdb-prod/TPC/Calib/PadGainFull/1546300800000/, curl response code:500
./digi.log[FATAL] Got nullptr from CCDB for path TPC/Calib/PadGainFull and timestamp 1546300800000
./digi.log[FATAL] unknown error while setting up workflow in o2-sim-digitizer-workflow.


## sw/BUILD/o2checkcode-latest/log
--
========== List of errors found ==========
++ GRERR=0
++ grep -a -v clang-diagnostic-error error-log.txt
++ grep -a ' error:'
++ GRERR=1
++ [[ 1 == 0 ]]
++ mkdir -p /sw/INSTALLROOT/fbe43246c9da31c91eae8a10eacd43845dc84459/slc9_x86-64/o2checkcode/1.0-local35/etc/modulefiles
++ alibuild-generate-module
+ HAS_BIN=
+ HAS_LIB=
+ HAS_CMAKE=
+ HAS_ROOT=
+ '[' 0 '!=' 0 ']'
+ cat
+ printf 'if ![ is-loaded '\''BASE/1.0'\'' ] {\n module load BASE/1.0\n}'
+ echo 'BUILD_REQUIRES=CMake alibuild-recipe-tools defaults-release'
BUILD_REQUIRES=CMake alibuild-recipe-tools defaults-release
+ FULL_BUILD_REQUIRES='alibuild-recipe-tools json-c FairCMakeModules CMake ninja UUID pytorch_cpuinfo Alice-GRID-Utils date O2-customization googlebenchmark ninja-fortran double-conversion MPFR defaults-release Python-modules-list bz2 safe_int'
+ echo 'FULL_BUILD_REQUIRES=alibuild-recipe-tools json-c FairCMakeModules CMake ninja UUID pytorch_cpuinfo Alice-GRID-Utils date O2-customization googlebenchmark ninja-fortran double-conversion MPFR defaults-release Python-modules-list bz2 safe_int'
FULL_BUILD_REQUIRES=alibuild-recipe-tools json-c FairCMakeModules CMake ninja UUID pytorch_cpuinfo Alice-GRID-Utils date O2-customization googlebenchmark ninja-fortran double-conversion MPFR defaults-release Python-modules-list bz2 safe_int
++ env
++ cut -f1 -d=
++ grep -v '^DEFAULT_'
++ grep -v PKGREVISION
++ grep REVISION
++ grep -v ALIBUILD_RECIPE_TOOLS
++ sed -e s/_REVISION//
+ for x in $(env | cut -f1 -d= | grep -v "^DEFAULT_" | grep -v PKGREVISION | grep -v ALIBUILD_RECIPE_TOOLS | grep REVISION | sed -e 's/_REVISION//')
++ eval 'echo $LIBFFI_REVISION'
+++ echo 14
+ REVISION_VALUE=14
++ eval 'echo $LIBFFI_VERSION'
+++ echo v3.2.1-alice1
+ VERSION_VALUE=v3.2.1-alice1
++ eval 'echo $LIBFFI_ROOT'
+++ echo /sw/slc9_x86-64/libffi/v3.2.1-alice1-14
+ ROOT_PATH_VALUE=/sw/slc9_x86-64/libffi/v3.2.1-alice1-14
+ '[' -z 14 ']'
[0 more errors; see full log]

Full log here.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@pkurash pkurash closed this Oct 22, 2026
@pkurash

pkurash commented Oct 22, 2026

Copy link
Copy Markdown
Contributor Author

New PR:
#15967

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@njacazio njacazio njacazio left review comments
@mconcas mconcas Awaiting requested review from mconcas mconcas is a code owner
@fcolamar fcolamar Awaiting requested review from fcolamar fcolamar is a code owner
@shahor02 shahor02 Awaiting requested review from shahor02 shahor02 is a code owner

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@pkurash @alibuild @vkucera @njacazio
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.