Skip to content

Add native Pegasus and Zephyr plot helpers - #62

Merged
bernalde merged 1 commit into
mainfrom
feature/issue-61-topology-plots
Jun 30, 2026
Merged

Add native Pegasus and Zephyr plot helpers#62
bernalde merged 1 commit into
mainfrom
feature/issue-61-topology-plots

Conversation

@bernalde

Copy link
Copy Markdown
Member

Closes #61

Summary

  • Add DWave.draw_topology for Pegasus, Zephyr, WorkingGraph, sampler metadata, sample sets, and sampler objects.
  • Add DWave.draw_embedding for returned embeddings over the full calibrated Pegasus/Zephyr working graph.
  • Return a notebook-displayable Matplotlib figure by default, while supporting caller-supplied ax values.
  • Document the plotting recipe and declare Matplotlib in the managed Python environment.

Tests

  • DWAVE_API_TOKEN= JULIA_DEPOT_PATH=/home/bernalde/repos/DWave.jl/.julia-depot julia --project=. --startup-file=no test/topology.jl
  • DWAVE_API_TOKEN= JULIA_DEPOT_PATH=/home/bernalde/repos/DWave.jl/.julia-depot julia --project=. --startup-file=no -e 'using Pkg; Pkg.test()'

The live DWave.Optimizer conformance tests were skipped because DWAVE_API_TOKEN was empty.

Branch Hygiene

  • Base branch: main
  • Source branch point: origin/main@a35fa52
  • Head branch: feature/issue-61-topology-plots
  • Stacked status: not stacked
  • Prerequisite PRs: none

@bernalde
bernalde marked this pull request as ready for review June 30, 2026 16:18

@bernalde bernalde left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No blocking findings from this review.

The implementation matches issue #61's core request: it adds native DWave.draw_topology and DWave.draw_embedding helpers, routes Pegasus and Zephyr working graphs through D-Wave NetworkX's canonical drawing functions, and keeps embedding overlays on the full calibrated working graph rather than only the used qubits. The metadata/sample-set overloads preserve the DWave.WorkingGraph(QUBOTools.metadata(sampleset)) path, and the README documents the notebook-facing usage.

Blocking issues: none.

Nonblocking issues: none.

Questions: none.

Tests run and outcomes:

  • DWAVE_API_TOKEN= JULIA_DEPOT_PATH=/home/bernalde/repos/DWave.jl/.julia-depot julia --project=. --startup-file=no test/topology.jl passed locally against e1f6a662081ff803a14abf236fd1500a941663b1.
  • GitHub CI is green for Julia 1.10 and Julia 1 on both Ubuntu and Windows.
  • Live D-Wave sampler coverage was not exercised locally because DWAVE_API_TOKEN is empty; the existing live-token test path is skipped in that state.

Merge-readiness: the PR is ready for another maintainer's formal approval. This review is posted as COMMENT because the authenticated GitHub account (bernalde) is also the PR author, so GitHub will not accept a formal approval from this account.

@bernalde
bernalde merged commit cb775b8 into main Jun 30, 2026
4 checks passed
@bernalde bernalde mentioned this pull request Jun 30, 2026
@bernalde
bernalde deleted the feature/issue-61-topology-plots branch June 30, 2026 18:52
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.

Add native Pegasus and Zephyr topology/embedding plot helpers

1 participant