search

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

Suggest using `EmbeddingComposite` when the hardware graph is incompatible #424

Closed AkashNarayanan closed 2 years ago

AkashNarayanan commented 2 years ago

Description of the Change: Updated the error message in sample_ising() method to suggest using EmbeddingComposite when the hardware graph is incompatible with the solver.

Related Issue: Closes #407

AkashNarayanan commented 2 years ago

Should the error message be updated in sample() method too?

codecov-commenter commented 2 years ago

Codecov Report

Merging #424 (89c96b4) into master (8272705) will decrease coverage by 1.38%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #424      +/-   ##
==========================================
- Coverage   91.82%   90.43%   -1.39%     
==========================================
  Files          22       22              
  Lines        1370     1370              
==========================================
- Hits         1258     1239      -19     
- Misses        112      131      +19
Impacted Files Coverage Δ
dwave/system/samplers/dwave_sampler.py 84.37% <0.00%> (-3.13%) :arrow_down:
dwave/system/samplers/clique.py 77.35% <0.00%> (-5.04%) :arrow_down:
dwave/system/samplers/leap_hybrid_sampler.py 75.71% <0.00%> (-2.86%) :arrow_down:
dwave/system/composites/embedding.py 95.85% <0.00%> (-1.19%) :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 8272705...89c96b4. Read the comment docs.