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

Updates for eiger stream2 / no fan / single file writer #135

Open
1 task
GDYendell opened this issue Aug 3, 2023 · 2 comments
Open
1 task

Updates for eiger stream2 / no fan / single file writer #135

GDYendell opened this issue Aug 3, 2023 · 2 comments

Comments

@GDYendell
Copy link

GDYendell commented Aug 3, 2023

In updating to odin to work with stream2 some simplifications have been made.

  • The receivers have been removed entirely
  • The fan is optional
  • There is an option to have 1 or 4 writers

I am hoping to use a single writer and no fan everywhere, but we need to verify that it is sufficient to handle the data rates. It may be easier to patch things enough to test a single writer and then if it works then decide to only support a single writer to simplify the code, rather than to update it to work in either case.

Changes I can see from reviewing the code:

  • Don't check for the frame receiver here
  • Don't check frames timed out here
  • I am going to try to override FramesDropped_RBV to be eiger specific but it might need to be a new PV if it isn't possible
  • I don't see any specific examples, but don't set AcquisitionID - I have disabled the record so that it is always empty and doesn't ignore any frames (and am now checking the series ID from the start message to make sure the image messages are in the expected acquisition)

And depending on if it needs to support both single writer and fan with 4 writers:

  • Use OD:NumProcesses_RBV to query only the writers that exist
  • Don't check the fan is initialised / connected here
@GDYendell
Copy link
Author

GDYendell commented Aug 23, 2023

From testing data throughput issues on I04, we might not be able to get rid of the fan + 4 FPs...

@stan-dot stan-dot added the i04 label Sep 25, 2024
@stan-dot
Copy link
Contributor

from discussion - this is blocked until the lower level eiger work is done

@stan-dot stan-dot added the eiger label Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants