Skip to content

[codex] Sync student-facing Notebook 4 (D-Wave)#11

Draft
bernalde wants to merge 1 commit intomainfrom
codex/sync-notebook4-dwave
Draft

[codex] Sync student-facing Notebook 4 (D-Wave)#11
bernalde wants to merge 1 commit intomainfrom
codex/sync-notebook4-dwave

Conversation

@bernalde
Copy link
Copy Markdown
Member

Closes #8.

Summary

This PR aligns the Python and Julia Notebook 4 materials at the student-facing level without changing the executable notebook logic.

What Changed

  • aligned the Notebook 4 section flow around the problem statement, example, quantum annealing section, and references
  • added Python-side environment and execution notes near the setup/import area
  • added explicit Python-side D-Wave account guidance around dwave setup, dwave ping, and DWaveSampler()
  • synchronized the QUBO notation in the problem statement and matrix-definition walkthrough
  • updated the Julia Notebook 4 metadata version so it matches the committed Julia environment
  • added a Notebook 4 pair-sync test module that checks headings, setup guidance, references, notation, and Julia metadata

Validation

  • /home/bernalde/repos/QuIP/.venv/bin/python -m unittest tests.test_notebook4_pair_sync
  • /home/bernalde/repos/QuIP/.venv/bin/python -m unittest discover -s tests
  • ~/.local/bin/uv run --group docs python -m unittest discover -s tests

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -91,9 +91,7 @@
"source": [
Copy link
Copy Markdown
Member Author

@bernalde bernalde Apr 17, 2026

Choose a reason for hiding this comment

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

This is lacking to mention the DWave.jl package and more importantly we need to make sure that it works with the latest version of the package


Reply via ReviewNB

@@ -91,9 +91,7 @@
"source": [
Copy link
Copy Markdown
Member Author

@bernalde bernalde Apr 17, 2026

Choose a reason for hiding this comment

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

We need to add more references here


Reply via ReviewNB

@@ -39,20 +39,32 @@
"metadata": {},
Copy link
Copy Markdown
Member Author

@bernalde bernalde Apr 17, 2026

Choose a reason for hiding this comment

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

If we are going to redefine the constant as beta, then the code below needs to also follow that as it still defines the variable as cQ


Reply via ReviewNB

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.

Sync Python and Julia student-facing content for Notebook 4 (D-Wave)

1 participant