Skip to content

Make the non-prompt injection ratio configurable also for p-O#2402

Merged
iarsene merged 1 commit into
AliceO2Group:masterfrom
ppoava:patch-1
Jul 9, 2026
Merged

Make the non-prompt injection ratio configurable also for p-O#2402
iarsene merged 1 commit into
AliceO2Group:masterfrom
ppoava:patch-1

Conversation

@ppoava

@ppoava ppoava commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Following the commit for O-O (see c53f5cd) from donglovo, the non-prompt injection ratio has now also made configurable for the p-O case.

I copy here below the description from the commit above, as this one is identical:

  • Set the non-prompt injection ratio to 2

Change the default injection ratio from 5 to 2 for the non-prompt gap-triggered generators, corresponding to one injected signal event every two generated events.

  • Allow input trigger ratio configuration via environment variable

Read INPUT_TRIGGER_RATIO from the environment and use it to override the default input trigger ratio. Fall back to the constructor default when the variable is unset or invalid.

Following the commit for O-O (see AliceO2Group@c53f5cd) from donglovo, the non-prompt injection ratio has now also made configurable for the p-O case.

I copy here below the description from the commit above, as this one is identical:

* Set the non-prompt injection ratio to 2

Change the default injection ratio from 5 to 2 for the non-prompt gap-triggered generators, corresponding to one injected signal event every two generated events.

* Allow input trigger ratio configuration via environment variable

Read INPUT_TRIGGER_RATIO from the environment and use it to override the default input trigger ratio. Fall back to the constructor default when the variable is unset or invalid.
@ppoava ppoava requested a review from a team as a code owner July 8, 2026 18:56
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@iarsene iarsene enabled auto-merge (squash) July 9, 2026 12:20
@iarsene iarsene merged commit 18f55b0 into AliceO2Group:master Jul 9, 2026
8 checks passed
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.

2 participants