Skip to content

ITS3: Base: Make ALPIDE pixel size configurable - #10690

Merged
mconcas merged 1 commit into
AliceO2Group:devfrom
f3sch:its3_segAlpideConf
Feb 7, 2023
Merged

ITS3: Base: Make ALPIDE pixel size configurable#10690
mconcas merged 1 commit into
AliceO2Group:devfrom
f3sch:its3_segAlpideConf

Conversation

@f3sch

@f3schf3sch commented Feb 3, 2023

Copy link
Copy Markdown
Collaborator

In preparation for evaluating digits and different pixel sizes.

Signed-off-by: Felix Schlepper f3sch.git@outlook.com

mconcas
mconcas previously approved these changes Feb 4, 2023

@fgrosafgrosa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@f3sch the code looks good, thanks a lot! I would just ask you to rename "Alpide" to "SuperAlpide" to be cosistent with the SegmentationSuperAlpide (this will not be the final name of the ITS3 chip, but still).

v1->v2: Changed Name from Alpide to SuperAlpide to be more consistent
Signed-off-by: Felix Schlepper <f3sch.git@outlook.com>
@mconcas
mconcas merged commit 06af269 into AliceO2Group:devFeb 7, 2023
@f3sch
f3sch deleted the its3_segAlpideConf branch February 7, 2023 09:59
Sign up for freeto 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

@f3sch@mconcas@fgrosa