-
Notifications
You must be signed in to change notification settings - Fork 9
Filters, shutters, and distance sensors #47
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
Conversation
…niblitz shutter controllers.
…cks. Added support for KeyenceCL3000 distance sensor.
.loglogin
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file should be deleted. It is created by scripts that are automatically run on APS linux machines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Common to add .loglogin to the repostory's .gitignore file. GitHub makes it possible to provide an organization-wide set of features (workflows, for example). The Bluesky team has been discussing adding a org-level .gitignore (in the BCDA-APS org) that would include .DS_Store, .loglogin, and some other common things one would not normally keep in a repo (such as .vscode).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change this:
iocsh/
To this:
/iocsh/
So that only the top-level, autogenerated iocsh dir is ignored.
kmpeters
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm comfortable merging this pull request.
|
@prjemian are there any other changes you think should be made before this PR is merged? |
prjemian
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes in .gitignore look right. I'm skipping review of the other changes.
Adding support for serial-based XIA PFCU filters; Keyence CL3000 distance sensor; Thorlabs SC-10 shutter controller; and Uniblitz shutter controllers.