sc-99549-fix-broken-demo-inter-sphinx-links-within-demo #1550
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
👋 Hey, looks like you've updated some demos! 🐘 Don't forget to update the Please hide this comment once the field(s) are updated. Thanks! |
ashishks0522
requested review from
Alan-eMartin,
jontoye and
doctorperceptron
Your preview is ready 🎉!You can view your changes here |
ashishks0522
changed the title
Merged
doctorperceptron
added a commit
that referenced
this pull request
Sep 19, 2025
This PR tweaks the `qml` build tool such that the generated objects.inv files match the naming and uris for the demos on pennylane.ai. This PR should fix #1550. Note: This has been tested locally. ~We should verify it works when building all demos as wel.~ Tested for all demos successfully [here](https://github.com/PennyLaneAI/qml/actions/runs/17866805882) --------- Co-authored-by: Alan Martin <alan.martin@xanadu.ai>
|
👋 Hey, looks like you've updated some demos! 🐘 Don't forget to update the Please hide this comment once the field(s) are updated. Thanks! |
|
Looks good @ashishks0522. Now we just need the html redirects and we're good. This can be merged in now. |
doctorperceptron
approved these changes
Sep 22, 2025
**Summary:** This PR reduces some of the image sizes for the `sc_qubits` demo to test out the impact on `pennylane.ai` page loading performance.
Alan-eMartin
deleted the
sc-99549-remove-html-from-demo-references-in-qml-demos
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
.htmlfrom demo references as we no longer need that in the URL and its causing 404 with next port, update it to be intersphinx formatTicket