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

IRIS: Support for MKS PDR2000 Pressure Transducer #1406

Closed
kjwoodsISIS opened this issue Jul 13, 2016 · 11 comments
Closed

IRIS: Support for MKS PDR2000 Pressure Transducer #1406

kjwoodsISIS opened this issue Jul 13, 2016 · 11 comments

Comments

@kjwoodsISIS
Copy link
Contributor

kjwoodsISIS commented Jul 13, 2016

As scientist working on IRIS, I need to monitor the read backs from a MKS PDR2000 Pressure Transducer (connected to a Baratron pressure valve).

Acceptance Criteria
  1. I can view the PVs which define the state of the PDR2000 Pressure Transducer.
  2. I can view the state of the PDR2000 Pressure Transducer via the IBEX GUI.
Notes
  1. We can base the IOC and OPI on the existing VI for the PDR2000 Pressure Transducer.
  2. There is an existing EPICS driver for a MKS PDR4000 Pressure Transducer - see http://www.aps.anl.gov/epics/modules/manufacturer.php#MKS%20Instruments - that could, perhaps, be adapted. Alternatively, it might be simpler to create our own driver from scratch.
  3. When we have created the IOC and OPI, ask if we can test with a real MKS PDR2000.
@AdrianPotter
Copy link

Waiting to hear from Gareth (or @KathrynBaker?) about getting hold of the VI and/or reference manual. Can't find it online. Putting back into pool until we've got a starting point.

@AdrianPotter AdrianPotter removed their assignment Jul 19, 2016
@kjwoodsISIS
Copy link
Contributor Author

There is an on-line manual for the PDR2000.
There are also EPICS drivers for the PDR4000 transducer, which we might be able to modify.
For both of these, please follow the links from https://github.com/ISISComputingGroup/IBEX/wiki/IRIS-Instrument-Details#noteMKS

@AdrianPotter AdrianPotter self-assigned this Jul 19, 2016
@AdrianPotter
Copy link

Super, thanks. Also got the VIs from @KathrynBaker

@AdrianPotter
Copy link

@davidkeymer has also sent me a summary of the commands in use:

MKS PDR2000 commands used in VI: v - read version; p - read pressures; u - read units; f - read full scales.  All straightforward enough, however I think you may need to do some clever formatting of the replies to each command as both channels are returned.

@davidkeymer and Ian Silverstone currently looking into borrowing a physical device.

@AdrianPotter
Copy link

PR4000 IOC not applicable that I can see. Device protocol completely different.

@kjwoodsISIS
Copy link
Contributor Author

Ah well. It was worth a try! :-)

@AdrianPotter
Copy link

The IOC and OPI are both ready now. I've tested with an emulated device (see #1418). I would prefer not to declare this done though until it's had a dry run on an actual device either in the office or out and about in the field. @davidkeymer, please let me know if and when you find one I can try.

@AdrianPotter
Copy link

Have contacted Chris Goodway on the advice of Ian Silverwood

@AdrianPotter
Copy link

Moving to review. Please test with simulation mode and emulators as no device is currently available. It has been noted that refinement of the protocol is likely to be needed following the first test session on Iris

@AdrianPotter
Copy link

Pull requests:

EPICS - https://github.com/ISISComputingGroup/EPICS/pull/38
IOC - ISISComputingGroup/EPICS-ioc#59
Support module/device emulator - Already on master
GUI - ISISComputingGroup/ibex_gui#299

@IsabellaRey
Copy link

Haven't tested the functionality yet. Could you please check the following before I continue:

  • in st. cmd files dbLoadRecords command, the MKSPDR200 macro should be fixed to be IOCNAME
  • in the support module, the protocol should be built into data/ (and so st.cmd updated to reflect this)
  • IOC name should be max 8 chars long

Optional: have SIM macros (see TPG26X IOC) in the st.cmd so the IOC can easily be started in RECSIM or DEVSIM mode

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants