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

Deprecate `VirtualGraphComposite` #532

Closed JoelPasvolsky closed 3 months ago

JoelPasvolsky commented 3 months ago

Closes https://github.com/dwavesystems/dwave-system/issues/531 and https://github.com/dwavesystems/dwave-system/issues/54

codecov[bot] commented 3 months ago

Codecov Report

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

Project coverage is 86.71%. Comparing base (b233941) to head (c4da9e1). Report is 9 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #532 +/- ## ========================================== - Coverage 89.65% 86.71% -2.94% ========================================== Files 24 24 Lines 1760 1762 +2 ========================================== - Hits 1578 1528 -50 - Misses 182 234 +52 ```

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

arcondello commented 3 months ago

We should update the tests with assertWarns for the deprecation warning. I know we already have a few warnings when we run the tests, but I'd like to not make it worse.