Skip to content

Try using regex to change qml to qp#1743

Open
drdren wants to merge 15 commits intomasterfrom
dren-regex-qml-qp1
Open

Try using regex to change qml to qp#1743
drdren wants to merge 15 commits intomasterfrom
dren-regex-qml-qp1

Conversation

@drdren
Copy link
Copy Markdown
Contributor

@drdren drdren commented Apr 10, 2026

Before submitting

Please complete the following checklist when submitting a PR:

  • Ensure that your tutorial executes correctly, and conforms to the
    guidelines specified in the README.

  • Remember to do a grammar check of the content you include.

  • All tutorials conform to
    PEP8 standards.
    To auto format files, simply pip install black, and then
    run black -l 100 path/to/file.py.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.


Title:

Summary:

Relevant references:

Possible Drawbacks:

Related GitHub Issues:


If you are writing a demonstration, please answer these questions to facilitate the marketing process.

  • GOALS — Why are we working on this now?

    Eg. Promote a new PL feature or show a PL implementation of a recent paper.

  • AUDIENCE — Who is this for?

    Eg. Chemistry researchers, PL educators, beginners in quantum computing.

  • KEYWORDS — What words should be included in the marketing post?

  • Which of the following types of documentation is most similar to your file?
    (more details here)

  • Tutorial
  • Demo
  • How-to

@drdren drdren requested review from a team as code owners April 10, 2026 17:15
@github-actions
Copy link
Copy Markdown

👋 Hey, looks like you've updated some demos!

🐘 Don't forget to update the dateOfLastModification in the associated metadata files so your changes are reflected in Glass Onion (search and recommendations).

Please hide this comment once the field(s) are updated. Thanks!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 10, 2026

Your preview is ready 🎉!

You can view your changes here

Deployed at: 2026-04-17 22:15:21 UTC

David and others added 2 commits April 10, 2026 18:23
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What happened to this demo specficially? Seems like GH regards the change here to be over the whole file. Is there any file encoding thing changed here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi @JerryChen97, broadly speaking, the only changes I should have made are:

  1. import pennylane as qml -> import pennylane as qp
  2. qml.XYZ -> qp.XYZ

I do not know why ensemble_multi_qpu is so unhappy. Let me investigate.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah similar issues happened in the past actually. I couldn't find the exactly link to PR's but from my memory they used to be file encoding difference.

Copy link
Copy Markdown
Contributor

@JerryChen97 JerryChen97 left a comment

Choose a reason for hiding this comment

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

Mind try these on those weird file changes? It should be a typical line ending mutation issue and hopefully adding newline="" prevents Python from normalizing the line endings

Comment thread change_qml_qp.py Outdated
Comment thread change_qml_qp.py Outdated
Copy link
Copy Markdown
Contributor

@JerryChen97 JerryChen97 left a comment

Choose a reason for hiding this comment

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

thanks!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What happened here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Probably a formatting error. It's easier for me to just delete this file from the PR than to remedy it here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

OK deleted the file

@drdren drdren requested a review from josh146 April 16, 2026 22:04
@drdren
Copy link
Copy Markdown
Contributor Author

drdren commented Apr 17, 2026

Hey @andrijapau , would you mind approving this PR if you are OK with it? @JerryChen97 and I want to see if it suffices or if a Content approval is required. Thanks :)

@drdren drdren requested a review from andrijapau April 17, 2026 19:29
@andrijapau
Copy link
Copy Markdown
Contributor

Hey @andrijapau , would you mind approving this PR if you are OK with it? @JerryChen97 and I want to see if it suffices or if a Content approval is required. Thanks :)

I'd prefer if someone from Content could take over for review - I have my hands full over in PennyLane 😓

@andrijapau andrijapau removed their request for review April 17, 2026 19:32
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.

4 participants