Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent!

Not related to this PR but relevant in terms of the docs: pinning docutils at 0.16 is not necessary now because the current sphinx_rtd_theme release is compatible with later versions. I removed the pin from requirements.txt and locally the result looks good to me, though I did not carefully compare it side-by-side with the current case (of docutils pinned at 0.16). If you think this is too risky to do now, OK to wait and make the change after releasing version 0.10.2.

If you have access to the ReadTheDocs admin panel, it is worth checking why a warning banner is not appearing on older versions, as I commented in a recent issue.

While reviewing, I noticed a misprint in the docs: "indexed indexed" appears twice in timeresp.py:

• Time evolution of the state vector, indexed indexed by either the

• Time evolution of the state vector, indexed indexed by either the