Skip to content

Instrument server documentation #93

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cynthianolan
Copy link
Contributor

adding documentation for the instrument server


`instruments`: this field will contain each instrument you wish to create. Under this field should be the names of each instrument you are creating

`type`: This specifies which QCoDeS driver the instrumentserver should use to create the instrument. Can either be a local file or a community-created driver
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it has to be the python import path. If they put a path this will not work. The only way to passing this to a file is if they have that file in a package that install in editable mode.


`type`: This specifies which QCoDeS driver the instrumentserver should use to create the instrument. Can either be a local file or a community-created driver

`address`: the tcp/ip address of the instrument on the same network as the instrumentserver
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is whatever the instrument uses as an address. Sometimes its a tcp/ip other times a usb adress others a serial port or even the serial number if that driver uses that


`address`: the tcp/ip address of the instrument on the same network as the instrumentserver

`port`: which port (of the instrument) to communicate with (can generally be found in the manual of the instrument)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not part of the defaults arguments. This is specific to that instrument and that is why you need to put it inside of the init field. You should also mention what the init field does.

Comment on lines +152 to +153
The instrumentserver also contains functionality for config files. These config files contain set-up for QCoDeS instruments. These config files can then be used by the instrumentserver
on start-up to have the desired instruments already created without having to manually add them after starting the server.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

repeated These config ... start on 2 sentences in a row.

I think the wording is a little confusing. The instruments are not already created, instead the server will instantiate those instruments at startup. There is a minor difference there.

parameter_manager:
type: instrumentserver.params.ParameterManager
initialize: True
gui:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add the parameter manager with the custom GUI in the example before so you can mention how the instrumentserver supports custom gui's for instruments that need to be specified like this.


![parameter_manager_3](img/param_manager_3.png)

The instrumentserver also supports loading parameters from a file. They can also be created manually either from the terminal or the GUI.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replace the They can also... to Parameters can also...

If you want to load parameters from file, then name of that file is important. Depending on the name it will either: not load the parameters, fully replace the parameters (add a warning about how they are all lost if that happens), make a new profile that you can switch to the other parameters there.

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

Successfully merging this pull request may close these issues.

2 participants