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

Incorporate autoscaling into VirtualGraphComposite #104

Open vgoliber opened 6 years ago

vgoliber commented 6 years ago

Current Problem Scaling must be done manually for VirtualGraphComposite, but autoscaling is available for FixedEmbeddingComposite and EmbeddingComposite.

Proposed Solution Add autoscale to VirtualGraphComposite in same manner as for the other composites.

Alternatives Considered Provide a separate autoscale method

Additional context

schnorea commented 5 years ago

Is this something that is being considered? I have been looking through the code but can't seem to find an example of how this scaling should be done. Any pointers until this can be added?

arcondello commented 5 years ago

Yes this is being considered. No timeline yet, though we have gotten this request in a few places so we will prioritise it as a feature. The feature added in https://github.com/dwavesystems/dimod/pull/372 might help in the mean time. As of writing this message though it has not be deployed.

arcondello commented 5 years ago

191 gets us part of the way there. However, at the moment we cannot use the `ScaleComposite` as an intermediate composite between the `VirtualGraphComposite` and a `DWaveSampler` without also explicitly specifying a chain strength because the method used to determine the chain strength only looks at the child sampler.