Kibble-Zurek Mechanism with Noise Mitigation

[k8culver]

Thanks @AndyZzzZzzZzz for your work on this feature!

I can't remember if there were additional updates we wanted to add, all I've changed is cleaning up the code a bit and fixing the tests.

@jackraymond is going to help add tests for Andy's work.

I will follow this PR with a PR updating the code structure, adding more thorough doc strings, etc, for now this is mostly just Andy's updates.

[thisac]

Some high-level comments (still have to review most of the code):

• The logo and the tab-buttons aren't aligned with the edge and jump around quite a bit when resizing the window. It'd look better if they where at a constant distance from the edges, I think.
• When running the ZNE simulation, curve fitting seems to happen after a couple of runs with different coupling strengths. This doesn't seem to work as it should. If the fit fails, there's a popup and everything is temporarily gone. If the fit finishes and a curve appears, the scaling seems to be off. It also disappears (being replaced with the sample data points) if switching view.
• The original demo has no default value for QPU, causing an error (instead of a user-friendly warning) if a run is started without a chosen QPU.

[thisac]

Comments on what these do?

[k8culver]

• The logo and the tab-buttons aren't aligned with the edge and jump around quite a bit when resizing the window. It'd look better if they where at a constant distance from the edges, I think.

The break point jumping behaviour is standard practice for web development that Dash has built in with their Container component. But I agree that the sides should be lined up so I'll remove the Container here.

[k8culver]

• The original demo has no default value for QPU, causing an error (instead of a user-friendly warning) if a run is started without a chosen QPU.

Yeah I agree, I wonder if @JoelPasvolsky has a suggestion on which would make a good default?

[JoelPasvolsky]

Yeah I agree, I wonder if @JoelPasvolsky has a suggestion on which would make a good default?

I'd suggest following Ocean, just using the default that would be selected by DWaveSampler() based o feature selection defaults

[k8culver]

• When running the ZNE simulation, curve fitting seems to happen after a couple of runs with different coupling strengths. This doesn't seem to work as it should. If the fit fails, there's a popup and everything is temporarily gone. If the fit finishes and a curve appears, the scaling seems to be off. It also disappears (being replaced with the sample data points) if switching view

@thisac I think I fixed this bug but let me know if you're still seeing it

[k8culver]

@JoelPasvolsky @jackraymond @thisac I've made quite a few updates but I believe we're ready for another round of reviews.