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

Copper mountain S5048 driver #78

Merged
merged 6 commits into from
Dec 21, 2021

Conversation

Saba-Mehshar
Copy link
Contributor

This is the driver for copper mountain S5048 network analyzer. It can connect to the instrument, but not responding to the commands. Can anyone review it please? Many thanks.

@Saba-Mehshar
Copy link
Contributor Author

Hi,
I have followed the HP 8753D Network Analyzer Qcodes driver script to write this driver, but have changed the SCPI commands as per the CMT S5048 programming manual.
I have been getting error with 305 & 306 lines because of this it does get failed to set the display format and am unable to update any parameters.

@jenshnielsen
Copy link
Collaborator

@Saba-Mehshar I think you are making it much more difficult for you trying to take an existing driver and modifying it.
There are going to be a number of things that do not apply. I would recommend that you start with a minimal driver.

E.g. start with the CMTS5048 class and add parameters to this one by one making sure that they work. You can take inspiration from the HP driver but coping and modifying it directly is unlikely to be successful

@Saba-Mehshar
Copy link
Contributor Author

@jenshnielsen Thanks for the suggestion. This is what I did initially - I had started with a NetworkAnalyser_S5048 class and added all the parameters to it. Everything was working perfectly except that I was not able to save and plot the data. Then I thought of going for the HP 8753D VNA and its not working as well.
Okay, I will now do as you said - start with a CMTS5048 class, and then modify it.
Hope it works this time. Thanks again..

@Akshita07
Copy link
Contributor

@Saba-Mehshar Have you been able to make the driver work? If you need any help do let us know.

@Saba-Mehshar
Copy link
Contributor Author

Hi @Akshita07
Sorry for the late response.
I had started with a class and added all the parameters to it and it was working fine. Then, I added a few more things to it but couldn't test it yet. I will update you when I go back to the lab and start working on it. Thank you very much for offering kind help!

@Akshita07
Copy link
Contributor

Hello @Saba-Mehshar ,

Did you get a chance to test the new implementation? Do reach out for any help.

@erikawa-e
Copy link

Hi,
I think you can use M5180 driver:
https://github.com/QCoDeS/Qcodes_contrib_drivers/tree/master/qcodes_contrib_drivers/drivers/CopperMountain

I have Copper Mountain S5045 and it is working for me to some extent.
(I had to erase
"assert isinstance(self.instrument, M5180)"
part in
def get_raw
)
Make sure that you change the frequency/power ranges.

@astafan8
Copy link
Contributor

astafan8 commented Dec 7, 2021

@Saba-Mehshar @erikawa-e shall we proceed with this PR or not?

@astafan8 astafan8 enabled auto-merge December 21, 2021 13:02
@astafan8 astafan8 merged commit af6ec17 into QCoDeS:master Dec 21, 2021
@astafan8
Copy link
Contributor

@Saba-Mehshar @erikawa-e I fixed a few small things in the driver and will merge it. If you or other users find things to improve or fix, just open new PRs with that :) this driver will be part of 0.12.0 release of qcodes contrib drivers

@Saba-Mehshar
Copy link
Contributor Author

Saba-Mehshar commented Dec 21, 2021 via email

@Saba-Mehshar Saba-Mehshar deleted the Saba-Mehshar-patch-1 branch January 17, 2022 19:40
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.

5 participants