Change TPG threshold value type from u16 to s16.#287
Merged
Conversation
MRiganSUSX
approved these changes
Apr 28, 2026
Contributor
MRiganSUSX
left a comment
There was a problem hiding this comment.
tested max limit at ehn1 and worked as expected.
(whether this should be a signed int is beyond me)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
The value type in configuration for TPG thresholds were incorrectly set to unsigned integer 16 instead of signed. The problem was notice at-scale with ProtoDUNE VD (NP02) when setting the permissible configuration maximum for
u16. Manually setting the value tos16maximum behaved appropriately and as expected for TPG.Configurations should generally not be using threshold values near the maximum, so it is not expected to break existing configurations.
Type of change
Testing checklist
dbt-build --unittest)pytest -s minimal_system_quick_test.py)dunedaq_integtest_bundle.sh)python -m pytest)pre-commit run --all-files)Check that trying to set a threshold value greater than
32767is not permitted. There is otherwise no TPG or Readout functionality change. This is purely a first-level configuration change.Further checks
dbt-build --lint, and/or see https://dune-daq-sw.readthedocs.io/en/latest/packages/styleguide/)(Indicate issue here: # (issue))