qiskit-community / qiskit-metal

Quantum Hardware Design. Open-source project for engineers and scientists to design superconducting quantum devices with ease.
https://qiskit-community.github.io/qiskit-metal/
Apache License 2.0
270 stars 201 forks source link

Bug fix lumped_capacitive.py #981

Closed vanitery closed 5 months ago

vanitery commented 5 months ago

What are the issues this pull addresses (issue numbers / links)?

Error when running lumped_capacitive.levels_vs_ng_real_units with do_plot = 1.

Did you add tests to cover your changes (yes/no)?

None. Default setting of the function is do_plots=0, which is the only reason this issue hasn't broken out before. Adjusting this single argument in a plt.plot() call should have no bearing on any other function.

Did you update the documentation accordingly (yes/no)?

No, since this is covered in the documentation of matplotlib.

Did you read the CONTRIBUTING document (yes/no)?

Yes, although this seems like such a tiny change, and most of the guidelines overkill in this case.

Summary

Changing plt.plot() argument from "LineWidth" to "linewidth" to avoid run error. Specifically the function levels_vs_ng_real_units inside the file lumped_capacitive.py.

Details and comments

Tiny fix for an aggravating bug.

CLAassistant commented 5 months ago

CLA assistant check
All committers have signed the CLA.

CLAassistant commented 5 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

vanitery commented 5 months ago

Yeah sorry about that my friend, I was confused so I accidentally closed it and then reopened it. This makes re-approval necessary and I apologize for the inconvenience.

I was just wondering what I had to do to implement the change now that you approved the pull request? It was still not merged. I am somewhat new to this unfortunately.

zlatko-minev commented 5 months ago

No problem at all! I understand the confusion. The reason for the delay in merging is that we have a testing process in place, and I needed to wait for the test to finish running before I can click the final "merge" button. It's a necessary step to ensure that everything is working as expected.

I appreciate your patience in this process. I didn't click it yet though because i was distracted with some other things. You can drop me a comment note to remind me here in the future. Thank you for your understanding, and I'll make sure to proceed with the merge as soon as the testing is complete.

zlatko-minev commented 5 months ago

For example:

image
vanitery commented 5 months ago

Seems good to go again.

zlatko-minev commented 5 months ago

Done!