dwavesystems / dwave-system

An API for easily incorporating the D-Wave system as a sampler, either directly or through Leap's cloud-based hybrid samplers
https://docs.ocean.dwavesys.com/
Apache License 2.0
87 stars 62 forks source link

Enhance support for Zephyr topology #448

Closed seatim closed 2 years ago

seatim commented 2 years ago

Overview of changes:

Note: not included in this PR are:

codecov-commenter commented 2 years ago

Codecov Report

Merging #448 (22415fe) into master (791f3b2) will decrease coverage by 3.96%. The diff coverage is 17.24%.

@@            Coverage Diff             @@
##           master     #448      +/-   ##
==========================================
- Coverage   90.52%   86.56%   -3.97%     
==========================================
  Files          22       23       +1     
  Lines        1520     1533      +13     
==========================================
- Hits         1376     1327      -49     
- Misses        144      206      +62     
Impacted Files Coverage Δ
dwave/system/coupling_groups.py 12.50% <12.50%> (ø)
dwave/system/samplers/clique.py 74.35% <23.07%> (-8.04%) :arrow_down:
dwave/system/samplers/leap_hybrid_sampler.py 61.42% <0.00%> (-14.29%) :arrow_down:
dwave/system/samplers/dwave_sampler.py 84.75% <0.00%> (-2.44%) :arrow_down:
dwave/system/composites/embedding.py 96.00% <0.00%> (-1.15%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 791f3b2...22415fe. Read the comment docs.

boothby commented 2 years ago

this should probably be a PR to minorminer, as the chimera.py and pegasus.py have been reduced to stubs importing from there.

edit: that was hasty; this comment only applies to the zephyr.py file.