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

Upgrade to qiskit_sphinx_theme 1.12 #224

Merged

Conversation

Eric-Arellano
Copy link
Contributor

The 1.12 theme release is focused on stability, including CSS fixes. See https://github.com/Qiskit/qiskit_sphinx_theme/releases/tag/1.12.0rc1 for the changelog.

This PR also:

  • Enables -j auto with Sphinx to have parallelism. This should result in a faster docs build.
  • Adds tox -e docs-clean

Comment on lines -43 to +39
"sphinxcontrib.jquery", # Remove when changing html_theme to qiskit_ecosystem.
"qiskit_sphinx_theme",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This will still activate jQuery. It's more future-proof because qiskit_sphinx_theme now has some functionality like optionally supporting Translations.

Comment on lines -48 to -51
[pycodestyle]
max-line-length = 100
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Unused

@Eric-Arellano
Copy link
Contributor Author

I think CI might be broken on main.

@DanPuzzuoli DanPuzzuoli merged commit 9c0b248 into qiskit-community:main Jun 1, 2023
@Eric-Arellano Eric-Arellano deleted the upgrade-to-theme-12 branch June 1, 2023 18:05
DanPuzzuoli pushed a commit to DanPuzzuoli/qiskit-dynamics that referenced this pull request Jun 8, 2023
DanPuzzuoli added a commit that referenced this pull request Jun 9, 2023
* Temporary upper bound on JAX version <=0.4.6 and update to DynamicsBackend tutorial (#210)

* Add links of API reference to tutorials and userguides (#212)

* Fix multiset ordering bug in perturbation module (#211)

Co-authored-by: Ian Hincks <[email protected]>

* Bump Sphinx Theme to 1.11 (#215)

* Bump Sphinx Theme to 1.11

* Also activate jquery

* Bug fix: Measurement properties automatic padding for DynamicsBackend initialization (#209)

Co-authored-by: Daniel Puzzuoli <[email protected]>

* bounding ipython version for compatibility with python 3.8 (#216)

* Update deploy_documentation.sh to deploy in ecosystem (#219)

* Bound Diffrax version (#226)

* Bounding diffrax and equinox versions. The latest versions require
the latest version of JAX, but due to an unresolved bug in JAX,
Dynamics is only compatible with jax<=0.4.6. This commit also
adds a release note stating exactly what versions of these packages
will work with the latest version of dynamics.

* Upgrade to qiskit_sphinx_theme 1.12 (#224)

* Update links to repo and documentation (#227)

* updating minor comments in pulse sim tutorial to remove confusion (#228)

* adding max step size argument to dynamics_backend tutorial, with explanation (#229)

* fixing typo

---------

Co-authored-by: Kento Ueda <[email protected]>
Co-authored-by: Ian Hincks <[email protected]>
Co-authored-by: Eric Arellano <[email protected]>
Co-authored-by: Arthur Strauss <[email protected]>
Co-authored-by: Luciano Bello <[email protected]>
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.

2 participants