zlatko-minev / pyEPR

Powerful, automated analysis and design of quantum microwave chips & devices [Energy-Participation Ratio and more]
https://pyepr-docs.readthedocs.io
Other
160 stars 219 forks source link

importing pyepr into metal during CI on Metal is failing. #115

Open priti-ashvin-shah-ibm opened 2 years ago

priti-ashvin-shah-ibm commented 2 years ago

When metal is using the new tag for pip, CI fails on Metal. https://github.com/Qiskit/qiskit-metal/runs/6617275435?check_suite_focus=true

In vscode, one can see the error which would happen when imported using pip image

This is from https://www.geeksforgeeks.org/default-arguments-in-python/ image

@nikosavola , would you like to update the syntax, or I will try to do it tomorrow and have you be the reviewer?

nikosavola commented 2 years ago

You can update the syntax 👍

nikosavola commented 2 years ago

Should this issue be closed as of #116 ?