Skip to content

Make the non-prompt injection ratio configurable via environment variable#2400

Open
donglovo wants to merge 2 commits into
AliceO2Group:masterfrom
donglovo:master
Open

Make the non-prompt injection ratio configurable via environment variable#2400
donglovo wants to merge 2 commits into
AliceO2Group:masterfrom
donglovo:master

Conversation

@donglovo

@donglovo donglovo commented Jul 3, 2026

Copy link
Copy Markdown

This PR updates the non-prompt gap-triggered generator to:

  • set the default input trigger ratio to 2;
  • allow overriding it through the INPUT_TRIGGER_RATIO environment variable;
  • fall back to the default value when the environment variable is unset or invalid.

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.
@donglovo donglovo requested a review from a team as a code owner July 3, 2026 14:29
@github-actions

github-actions Bot commented Jul 3, 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

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.
@donglovo donglovo changed the title Set the non-prompt injection ratio to 2 Make the non-prompt injection ratio configurable via environment variable Jul 6, 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.

1 participant