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

Add feature flag `hss_solver_config_override` #427

Closed randomir closed 2 years ago

randomir commented 2 years ago

Add support for feature flags via DWAVE_FEATURE_FLAGS env var.

Re-introduce the #363 bug in spirit (hybrid solvers ignore solver def from config file/end), enabled by setting hss_solver_config_override feature flag.

This is to support LeapIDE upgrade.

codecov-commenter commented 2 years ago

Codecov Report

Merging #427 (ba9340a) into master (d5c06aa) will decrease coverage by 1.24%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #427      +/-   ##
==========================================
- Coverage   91.82%   90.58%   -1.25%     
==========================================
  Files          22       22              
  Lines        1370     1391      +21     
==========================================
+ Hits         1258     1260       +2     
- Misses        112      131      +19     
Impacted Files Coverage Δ
dwave/system/samplers/leap_hybrid_sampler.py 76.71% <100.00%> (-1.86%) :arrow_down:
dwave/system/utilities.py 100.00% <100.00%> (ø)
dwave/system/samplers/clique.py 77.35% <0.00%> (-5.04%) :arrow_down:
dwave/system/samplers/dwave_sampler.py 84.37% <0.00%> (-3.13%) :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 d5c06aa...ba9340a. Read the comment docs.