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 61 forks source link

Fix DWaveCliqueSampler integration tests #508

Closed arcondello closed 7 months ago

arcondello commented 8 months ago

Previously the tests would fail if there were other warnings raised.

codecov[bot] commented 8 months ago

Codecov Report

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

Comparison is base (2e70d8e) 87.70% compared to head (ad99d61) 87.74%. Report is 2 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #508 +/- ## ========================================== + Coverage 87.70% 87.74% +0.04% ========================================== Files 24 24 Lines 1675 1673 -2 ========================================== - Hits 1469 1468 -1 + Misses 206 205 -1 ```

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

arcondello commented 8 months ago

I am toying with just removing the warning altogether. I am not sure it's adding much benefit to users.

arcondello commented 8 months ago

Yeah, I think removing the warning is better. It's not really an actionable warning and the scaling event is desired behavior anyway. Went ahead and did that in https://github.com/dwavesystems/dwave-system/pull/508/commits/ad99d613465dbfb4710272d748f55f3942f9fbb1.