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

FrameReceiverAdapterController should have _unique_config set #46

Open
GDYendell opened this issue Oct 22, 2024 · 1 comment
Open

FrameReceiverAdapterController should have _unique_config set #46

GDYendell opened this issue Oct 22, 2024 · 1 comment
Assignees
@GDYendell
Labels
bug Something isn't working

Comments

@GDYendell
Copy link
Contributor

GDYendell commented Oct 22, 2024 •
edited
Loading

This will prevent creating config fan out PVs for configs that should be unique between processes, e.g. ctrl_endpoint

FrameProcessorAdapterController does this here.
@GDYendell GDYendell added the bug Something isn't working label Oct 22, 2024
@GDYendell GDYendell moved this to Current Sprint in Eiger / Odin FastCS Development Oct 22, 2024
@GDYendell GDYendell changed the title FrameReceiverAdapter should have _unique_config set FrameReceiverController should have _unique_config set Oct 22, 2024
@GDYendell GDYendell changed the title FrameReceiverController should have _unique_config set FrameReceiverAdapterController should have _unique_config set Oct 22, 2024
@GDYendell
Copy link
Contributor Author

I think it would make sense to have _unique_config on each specific class, rather than on the root adapter controller.

@GDYendell GDYendell self-assigned this Nov 29, 2024
@GDYendell GDYendell moved this from Current Sprint to In Progress in Eiger / Odin FastCS Development Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GDYendell GDYendell

Labels
bug Something isn't working
Projects
Eiger / Odin FastCS Development
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GDYendell