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.

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

Rest daq #35

Draft
wants to merge 43 commits into
base: master
Choose a base branch
from
Draft

Rest daq #35

wants to merge 43 commits into from

Conversation

juanangp
Copy link
Member

@juanangp juanangp commented Aug 17, 2021

juanangp Large: 553 Powered by Pull Request Badge

Changes for new restDAQ package.

  • Changes on TRestRawDAQMetadata
  • New constuctor for TRestRawSignal

Check https://github.com/rest-for-physics/restDAQ for more info on restDAQ package.

@juanangp juanangp requested review from jgalan and a team August 17, 2021 09:54
@jgalan jgalan requested a review from nkx111 August 17, 2021 10:36
@jgalan
Copy link
Member

jgalan commented May 24, 2022

What about this PR?

@juanangp
Copy link
Member Author

What about this PR?

I found issues while taking data with AlphaCAMM, the files gives error in the streamer while readint TRestRawDAQMetadata.

I have to understand what is going on...

@jgalan
Copy link
Member

jgalan commented May 30, 2022

This branch is 172 commits behind master. Latest master branch should be merged to this branch.

@juanangp
Copy link
Member Author

I know, I am working on it

@jgalan
Copy link
Member

jgalan commented May 30, 2022

Ok, I just got notified "ready to review" then I jumped into it.

@lobis lobis self-requested a review October 26, 2022 10:39
pre-commit-ci bot and others added 26 commits May 31, 2023 17:20
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.

4 participants