Skip to content

Fix integration test, no longer producing ADC patterns for CRT - #337

Open
xinyue-uoft wants to merge 1 commit into
developfrom
theo/crt_patterngen_fix
Open

Fix integration test, no longer producing ADC patterns for CRT#337
xinyue-uoft wants to merge 1 commit into
developfrom
theo/crt_patterngen_fix

Conversation

@xinyue-uoft

@xinyue-uoft xinyue-uoft commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

See DUNE-DAQ/datahandlinglibs#145 . datahandlinglibs function generate_periodic_adc_pattern is currently trying to distribute ADC pattern over 64 frames over CRT which has only 32 frames. Only thing preventing an runtime error is a blind try-catch statement. The correct behaviour should be that ADC pattern generation for CRTs should not be there in the first place. This PR changes it at integration test config substitution level to disable ADC pattern generation on CRT.

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature or enhancement (non-breaking change which adds functionality)
  • Optimization (non-breaking change that improves code/performance)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Testing checklist

  • Unit tests pass (e.g. dbt-build --unittest)
  • Minimal system quicktest passes (pytest -s minimal_system_quick_test.py)
  • Full set of integration tests pass (dunedaq_integtest_bundle.sh)
  • Python tests pass if applicable (e.g. python -m pytest)
  • Pre-commit hooks run successfully if applicable (e.g. pre-commit run --all-files)

Further checks

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