Skip to content
This repository has been archived by the owner on Aug 19, 2023. It is now read-only.

Fix tutorials default thumbnail #1708

Merged
merged 5 commits into from
Apr 2, 2023
Merged

Fix tutorials default thumbnail #1708

merged 5 commits into from
Apr 2, 2023

Conversation

HuangJunye
Copy link
Contributor

@HuangJunye HuangJunye commented Mar 28, 2023

Summary

nbsphinx 0.9 release changed the behaviour of the default thumbnail of jupyter notebook galleries. This PR revert its behaviour back to default as qiskit logo when no explicit thumbnail is set in the notebook.

Fixes Qiskit/qiskit#1707

Details and comments

Other changes are removing redundant optimization tutorials that have been moved to qiskit-optimization a long time ago.

Screenshots

After

image

Before

image

Comment on lines -166 to -82
'tutorials/optimization/1_quadratic_program':
'_static/optimization/1_quadratic_program.png',
'tutorials/optimization/2_converters_for_quadratic_programs':
'_static/optimization/2_converters.png',
'tutorials/optimization/3_minimum_eigen_optimizer':
'_static/optimization/3_min_eig_opt.png',
'tutorials/optimization/4_grover_optimizer':
'_static/optimization/4_grover.png',
'tutorials/optimization/5_admm_optimizer':
'_static/optimization/5_ADMM.png',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Optimization tutorials have been moved to qiskit-optimizations a long while ago.

Copy link
Collaborator

@Eric-Arellano Eric-Arellano left a comment

Choose a reason for hiding this comment

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

Thanks!

@1ucian0 1ucian0 added the automerge This PR will automatically merge once its CI has passed label Apr 2, 2023
@1ucian0 1ucian0 added the documentation Something is not clear or error in description label Apr 2, 2023
@mergify mergify bot merged commit ab1049a into Qiskit:master Apr 2, 2023
jakelishman pushed a commit to jakelishman/qiskit-terra that referenced this pull request Aug 11, 2023
* set default qiskit logo image

* remove optimization tutorials thumbnails

---------

Co-authored-by: Luciano Bello <bel@zurich.ibm.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge This PR will automatically merge once its CI has passed documentation Something is not clear or error in description
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing default thumbnail for tutorials after nbsphinx 0.9 release
3 participants