-
Notifications
You must be signed in to change notification settings - Fork 318
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
docs: Write bigger picture #232
Comments
Yes, I would love to see a big picture explanation on the qcodes magics 😍 I guess @alexcjohnson is the only one who can do this at the moment? |
I'm hoping @giulioungaretti can write it with me kibitzing, that way he learns the code better, we get a chance to talk about what might want to change, and we make sure that the explanations are clear to at least 2 people. |
yeah @alexcjohnson that is a good idea. Maybe it would be nice if you could start with an overview of the current design main ideas, so it's also easier to just start diving into the code for everybody. |
Actually, I would say that I will take for ever for me to figure out a lot of whys just looking at the code. |
First list of why for @alexcjohnson
General:
ps, Some of these whys could also be extracted from the code, but I think it neater if we start with this maybe dumber/verbose approach and slim it down later |
re: instrument/parameterA Why a class rather than a property: A re: instrument/functionA
re: Command
|
Nice! Will start expanding on this soon.
|
Also, do you think you can sketch a class diagram of some sort ? |
Alright, let me try to describe some of these data and control flows. Control flow when you create an
|
Control and data flow when you run a
|
@alexcjohnson re: Instrument, do you think you could write a little extra after the proxy feature was added ? Like any gotchas, or what gets proxies where ? (kind of a slightly longer of the pull request message in #244. |
We definitely need to fill in the doc in the guide before adding features or things.
The reason for p1 is that it can be hard to figure out by using qcodes, what things do and why they are here or implement a certain way.
See f.ex #223.
The text was updated successfully, but these errors were encountered: