Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DataHandlingModel with IDT #362

Merged
merged 12 commits into from
Nov 21, 2024
Merged

DataHandlingModel with IDT #362

merged 12 commits into from
Nov 21, 2024

Conversation

denizergonul
Copy link
Contributor

@denizergonul denizergonul commented Nov 8, 2024 •
edited
Loading

  • TriggerDataHandlingModel is introduced, which derives from datahandlinglibs::DataHandlingModel and overrides transform_payload in a way that IDT is properly transformed to RDT (depending on what IDT is).

  • TPSetSourceModel and TriggerSourceModel are replaced by datahandlinglibs::DataSubscriberModel.

datahandlinglibs pull request
appmodel pull request
fdreadoutlibs pull request

JIRA task

This was referenced Nov 8, 2024
Merged
Merged
Merged
@denizergonul denizergonul self-assigned this Nov 11, 2024
@ArturSztuc ArturSztuc mentioned this pull request Nov 18, 2024
Closed
@ArturSztuc ArturSztuc linked an issue Nov 18, 2024 that may be closed by this pull request
OKS: Change TriggerSourceModel so it inherits from DataSubscriberModel and overrides handle_payload #281
Closed
@denizergonul denizergonul mentioned this pull request Nov 21, 2024
Merged
@denizergonul denizergonul merged commit 3cbbc8e into develop Nov 21, 2024
1 check failed
@denizergonul denizergonul deleted the dte/datahandlingmodel_with_idt_tpa_vector branch November 21, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@roland-sipos roland-sipos roland-sipos approved these changes

@glehmannmiotto glehmannmiotto Awaiting requested review from glehmannmiotto

Assignees

@denizergonul denizergonul

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

OKS: Change TriggerSourceModel so it inherits from DataSubscriberModel and overrides handle_payload
2 participants
@denizergonul @roland-sipos