Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrected and added sections in Floquet Notebook. #121

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CamilleLCal
Copy link

  1. Removed unnecessary variables in some functions.
  2. Removed wf_gnd_prob in cells 6 to 11 because equivalent to f_gnd_prob.
  3. Added comments on the Floquet spectrum folding.
  4. Added integration options in some functions (possible with the new version of floquet.py)
  5. Some objects needed to be converted to Qobjs.
  6. Modified mesolve to sesolve in the unitary dynamics example to avoid confusion with dissipative dynamics.
  7. Comment on the Floquet-Markov ME about what result to expect in some specific cases.
  8. The example about the dissipative dynamics what modified a lot. We modified the c_ops and the noise spectrum J_cb in order to be able to compare the Lindblad ME to the FM-ME. The basis change was modified to match the new version of floquet_markov_mesolve which now integrates the master equation in the interaction picture. Notice that the previous version of this example was only working for some specific choice of parameters but there was an error in floquet_master_equation_tensor. Added a 6-th method with the fmmesolve function. The new example computes 3 expectation values.
  9. Added an example for the Floquet matrix elements X and compare them to analytics for a driven-dissipative two level system. Plot the matrix elements as a function of the drive amplitude.
  10. The other examples were not checked.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant