Skip to content

Update learning back button #4352

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.

Sign up for GitHub

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

Jump to bottom
Open
christopherporter1 wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update learning back button #4352

christopherporter1 wants to merge 2 commits into from

Conversation

@christopherporter1
Copy link
Collaborator

@christopherporter1 christopherporter1 commented Dec 4, 2025

Fixes #3883

@review-notebook-app
Copy link

review-notebook-app bot commented Dec 4, 2025

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@christopherporter1 christopherporter1 marked this pull request as ready for review December 4, 2025 15:53
@qiskit-bot
Copy link
Contributor

qiskit-bot commented Dec 4, 2025

Thanks for contributing to Qiskit documentation!

Before your PR can be merged, it will first need to pass continuous integration tests and be reviewed. Sometimes the review process can be slow, so please be patient. Thanks! 🙌

One or more of the following people are relevant to this code:

@Eric-Arellano Eric-Arellano changed the title Cp/mod toc Update learning back button Dec 4, 2025
Eric-Arellano
Eric-Arellano reviewed Dec 4, 2025
View reviewed changes
learning/courses/quantum-machine-learning/classical-ml-review.ipynb
Copy link
Collaborator

@Eric-Arellano Eric-Arellano Dec 4, 2025

Choose a reason for hiding this comment

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

Is this intentional? The image file name is unsup but this talks about supervised learning.

Copy link
Collaborator Author

@christopherporter1 christopherporter1 Dec 8, 2025

Choose a reason for hiding this comment

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

@Eric-Arellano Do you mean "qml-cr-background-sup-unsup.avif"? It is a comparison of supervised and unsupervised. So it has "sup" and "unsup" in the title.

learning/courses/quantum-machine-learning/introduction.ipynb
Copy link
Collaborator

@Eric-Arellano Eric-Arellano Dec 4, 2025

Choose a reason for hiding this comment

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

Ditto if this is intentional

Copy link
Collaborator Author

@christopherporter1 christopherporter1 Dec 8, 2025

Choose a reason for hiding this comment

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

@Eric-Arellano Ditto.

@christopherporter1
Copy link
Collaborator Author

christopherporter1 commented Dec 8, 2025

@kaelynj Could you look at the changes to text in the notebooks ml-review and introduction
I accidentally updated the figure descriptions in this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Eric-Arellano Eric-Arellano Eric-Arellano left review comments

@kaelynj kaelynj Awaiting requested review from kaelynj

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

Docs Planning
Status: No status

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update the learning back button to use our new landing pages

3 participants

@christopherporter1 @qiskit-bot @Eric-Arellano