search

epics-motor / motorAutomation1

EPICS motor drivers for Aerotech Automation1 controllers
Other
2 stars 1 forks source link

Bug fixes and suggested improvements #7

Closed ajgosl closed 2 years ago

ajgosl commented 3 years ago

These changes are suggested as a result of our tests of the support module.

rakaerotech commented 2 years ago

I think this looks good from Aerotech's perspective. I'm not sure if @kmpeters is interested in vetting the EPICS side of things.

kmpeters commented 2 years ago

Thanks for poking me. I didn't get an email notification when this pull request was created. I've updated my github notification settings so that shouldn't happen in the future.

I'd like to test this before it is merged.

kmpeters commented 2 years ago

I'm going to change the paths in motor.substitutions.automation1 after merging this pull request so that the example IOC still finds the databases when motorAutomation1 is built outside of the motor module.

Edit: The change is here: https://github.com/epics-motor/motorAutomation1/commit/a9719e98adfd988b371f486942d649ac81847a79