Skip to content

Specify strax mailbox processor in bootstrax#1577

Closed
cfuselli wants to merge 4 commits intomasterfrom
bootstrax-mailbox
Closed

Specify strax mailbox processor in bootstrax#1577
cfuselli wants to merge 4 commits intomasterfrom
bootstrax-mailbox

Conversation

@cfuselli
Copy link
Copy Markdown
Member

@cfuselli cfuselli commented Mar 7, 2025

This change is needed after AxFoundation/strax#904 that sets default processor to single thread. At the DAQ we want to keep mailbox, so we need to specify it now in bootstrax.

@cfuselli cfuselli requested a review from dachengx March 7, 2025 09:32
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 7, 2025

Coverage Status

coverage: 89.67% (+0.02%) from 89.649%
when pulling ac76143 on bootstrax-mailbox
into 9a9e3c7 on master.

not sure why but gives an error, likely due to the change in how the api is structured in the package
@cfuselli
Copy link
Copy Markdown
Member Author

cfuselli commented Mar 2, 2026

Close in favor of #1643

@cfuselli cfuselli closed this Mar 2, 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.

2 participants