-
Notifications
You must be signed in to change notification settings - Fork 2
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
CHRONUS: Use Multiple CAENV895 Crates #8357
CHRONUS: Use Multiple CAENV895 Crates #8357
Comments
How to review? can we test in any way or is this just code review? |
just code review, additionally this is already patched onto CHRONUS as part of testing so you can see the OPI doing its thing if you point it at the IOC on there. |
cool, thanks. is it worth adding argus and chronus to https://github.com/ISISComputingGroup/IBEX/wiki#instrument-information--hotfixes and listing hotfixes, in case something doesn't get merged before we release to them again? |
sure, I'll look at that |
As an instrument scientist, I want to be able to use multiple CARNV895 crates at once. The IOC and vendor library are theoretically able to load an arbitrary number of crates, but our startup needs to be altered to allow this. The vendor librry also requires certain id numbers to be passed to this, which should be configurable via macro
Acceptance Criteria
What is the acceptance criteria?
How to Test
verbose instructions for reviewer to test changes
(Add before making a PR)
The text was updated successfully, but these errors were encountered: