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

Support and abstract EPICS 7 pvAccess and pvData #212

Open
robnagler opened this issue Nov 20, 2024 · 3 comments
Open

Support and abstract EPICS 7 pvAccess and pvData #212

robnagler opened this issue Nov 20, 2024 · 3 comments

Comments

@robnagler
Copy link
Collaborator

robnagler commented Nov 20, 2024

utils.PV.caget/put should be generally named: get/put? Perhaps this is a per-device configuration, but probably shouldn't be in the interface directly.

@nneveu
Copy link
Member

nneveu commented Nov 22, 2024

@robnagler are you talking about this file? I think this is an example where things on the sc side were way ahead of the general device stuff. Maybe we can get rid of the pyepics folder and incorporate this in another place. @lisazacarias you wrote this and use this in some of your sc repos right? Maybe we can combine/move this to the device code area? Or is there a context where we would use this outside of interacting with devices?

@robnagler
Copy link
Collaborator Author

Yes, and there are a few other places (in bmad_modeling). Maybe channel access is necessary for now. It's use should be encapsulated so that the code doesn't have to change when channel access goes away.

@nneveu
Copy link
Member

nneveu commented Nov 22, 2024

Yes, definitely agree we need to merge/update all places using channel access to one location to prepare for the change. I think @williamColocho 's bmad_modeling functions will be easier to update and touch less things. Bmad and utils file changes could be separate issues.

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