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
90 stars 64 forks source link

Fix inconsistent sampleset interface across hybrid samplers #544

Closed randomir closed 4 weeks ago

randomir commented 4 weeks ago

Make sure sampleset returned always has a working wait_id() method.

The fix was actually implemented in https://github.com/dwavesystems/dimod/pull/1393 (released in dimod 0.12.18), and supported with https://github.com/dwavesystems/dwave-cloud-client/pull/674 in cloud-client.

On the dwave-system level, it's enough to require the latest dimod.

Fix #540.

codecov[bot] commented 4 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.93%. Comparing base (053d256) to head (e2e64b9). Report is 3 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #544 +/- ## ======================================= Coverage 88.93% 88.93% ======================================= Files 20 20 Lines 1582 1582 ======================================= Hits 1407 1407 Misses 175 175 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.