qiskit-metal icon indicating copy to clipboard operation
qiskit-metal copied to clipboard

Impedance & S analysis: Add fitting routine for resonance

Open zlatko-minev opened this issue 3 years ago • 10 comments

Please be as descriptive as possible, including: what is expected, why is this feature needed, what is the objective, etc.

What is the feature being requested?

Add a procedure to fit for a scattering curve S21 (complex), as discussed say here https://arxiv.org/pdf/2106.06173.pdf

image

What are use cases for this feature?

zlatko-minev avatar Jun 16 '21 12:06 zlatko-minev

Hello! I'm a new contributor but I'd like to try this one if that's ok. Khalil et.al. https://arxiv.org/abs/1108.3117 have a kind of similar discussion on microwave resonators, and lmfit uses their equation in a tutorial on fitting a complex resonator model https://lmfit.github.io/lmfit-py/examples/example_complex_resonator_model.html. We could adapt that approach to fit eq 45?

JoshDumo avatar Jun 30 '21 23:06 JoshDumo

Hi Josh,

This is great to hear. Yes, happy to have you contribute and you can even because an author on Metal with contributions.

Feel free to reach out on Slack in #metal channel if you run into issues.

Also relevant discussion may be https://qiskit.slack.com/archives/C01R8KP5WP7/p1626283484499800

zlatko-minev avatar Jul 15 '21 18:07 zlatko-minev

Hi @JoshDumo , how is this one going? Did you have any bandwidth to work on it?

marcolincs avatar Sep 15 '21 19:09 marcolincs

Hi @marcolincs I have a done a little bit but so far not so good.

Please have a look at some notebooks I have worked on so far, here in my github repository

As you'll see, the fitting doesn't work great just yet.

If you have any insights, I am very happy to hear them.

JoshDumo avatar Sep 17 '21 18:09 JoshDumo

Hi @JoshDumo Do you mind if I also try my hand at this? I have done this thing in C (https://github.com/SamarthH/phiRM-DCM). I think that I might be able to help here. If you want to work on this, you might want to look at appendix E of Jiansong Gao's thesis (the person who in my knowledge first described the phiRM)

SamarthH avatar Dec 02 '21 19:12 SamarthH

Yes, need a hand. Please go ahead.

JoshDumo avatar Dec 02 '21 21:12 JoshDumo

Awesome, you can post questions and updates here

zlatko-minev avatar Dec 02 '21 22:12 zlatko-minev

@zlatko-minev : I have implemented the methodology in pull request #737 . I have tested it with the output of Tutorial 4.16. Please let me know if any changes are necessary

SamarthH avatar Dec 06 '21 13:12 SamarthH

Great work!! We will look it over and commnet!

zlatko-minev avatar Dec 07 '21 17:12 zlatko-minev

Hi. Is there any further work left on this?

SamarthH avatar Dec 20 '21 13:12 SamarthH

Looks like work has merged. Will close this issue.

priti-ashvin-shah-ibm avatar Feb 01 '23 21:02 priti-ashvin-shah-ibm