-
Notifications
You must be signed in to change notification settings - Fork 320
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
Config System #341
Comments
This will solve most problems with the tight coupling of qcodes and jupyter. |
@MerlinSmiles @eendebakpt do you two, biggest advocates of config, have preferences? Others, please also say if you have preferences. I would not use a python file as it violates the principle of separating configuration data from code, which in the long run proves to be an effective way to avoid problems. |
No strong preferences. Either text (.ini) based or json is fine. No binary and no executable statements. |
I vote for json :) so much easier to work with than ini, the only downside is that it wont support comments... |
Proposed config system, looks like what most python packages have:
I know @eendebakpt has a nice basic PR in #302, and the proposed solution looks a lot like that one.
The text was updated successfully, but these errors were encountered: