qkitgroup / qkit

Qkit framework
GNU General Public License v2.0
43 stars 46 forks source link

QFIT always outputs its config on instanciation #70

Closed fr34q closed 5 years ago

fr34q commented 5 years ago

https://github.com/qkitgroup/qkit/blob/826f523efbb1f5f7d67da927910a0833628e7bfc/qkit/analysis/qfit.py#L111

I am using QFIT in headless mode in a script and would appreciate if this print statement can be turned off, e.g. if it would be conditional on some input parameter of the class (if it is necessary at all).

Any objections?

JBraumueller commented 5 years ago

No, please do! I also was about to make the unit conversion to GHz optional. Will you edit the python 2.7 version only or also the one in the python 3 branch?

fr34q commented 5 years ago

@JBraumueller Done. I also think it's a good idea to use SI units by default.