Closed tonyp-zi closed 2 years ago
I'd like to make a strong case to not create a SHF driver by a copy/paste/edit of the UHF driver, because it will be a nightmare to maintain and integrate into the rest of PycQED. IMHO it would be highly desirable to identify the commonalities between the instruments, and handle those in common code.
I'm actually working on splitting of hardware dependencies throughout PycQED (PR #675), and interaction with the UHF happens in many places (hopefully to be drastically reduced in the future). Making the interface of the UHF and SHF as common as possible/sensible, and making the commonalities explicit, will be some extra work now, but will probably be paid back many times during integration, debugging and maintenance.
@tonyp-zi this PR was deleted while I was typing the comment above, so the comment may no longer be valid
Hi @wvlothuizen, thank you for your quick feedback. That was just a very rough PR, to do some preliminary testing.
Your comment is very sensible, we will make sure to structure the driver properly to facilitate its maintenance. We still have to figure out exactly what can be shared between the two devices, and how to reflect it in the code, I will keep you updated.
@martinr-zi @mariosk-zi
Work in progress. I have just set up SHFQA files as copies of their UHFQA counterparts, and done some minor adjustements.
Todo: