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

Separate monitoring, event, and subarray/mcheader data in the eventsource #1040

Closed
watsonjj opened this issue Apr 5, 2019 · 4 comments
Closed

Comments

@watsonjj
Copy link
Contributor

watsonjj commented Apr 5, 2019

No description provided.

@watsonjj
Copy link
Contributor Author

watsonjj commented Apr 5, 2019

Current hierarchy:

  • event.mc
  • event.subarray
  • event.mcheader
  • event.mon
  • event....

Suggestion: Have a higher level container which is passed between pipeline algorithms, which contain different sub containers which are updated at different rates:

  • data.event (changes per event)
  • data.mon (changes every second or so)
  • data.header (changes every run/observation)
  • data.subarray (or include in data.header)
  • data....

@kosack
Copy link
Contributor

kosack commented Apr 8, 2019

I think we should make this a more general issue, since it affects more than the data model. I had started another one a while ago, but I think parts were also scattered in various PRs..:

See for example: #747

@kosack
Copy link
Contributor

kosack commented Apr 8, 2019

strangely I can't find the original discussion in the issue tracker... It was a general discussion on data access in ctapipe, and I had suggested exactly this splitting, but gave a few other options

@kosack
Copy link
Contributor

kosack commented Apr 8, 2019

I'm closing this one since it's replaced with #1041

@kosack kosack closed this as completed Apr 8, 2019
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

No branches or pull requests

2 participants