dwave-system icon indicating copy to clipboard operation
dwave-system copied to clipboard

Adding option to not use `exteded_j_range` in `DWaveCliqueSampler`

Open alexzucca90 opened this issue 4 years ago • 3 comments

Current Problem The clique sampler by default uses extended J range (when possible), disabling the ability to perform spin reversal transformations.

Proposed Solution if the parameter num_spin_reversal_transforms is passed to the sample method, we should use j_range instead of extended_j_range and maybe raise a warning.

Alternatives Considered The alternative is using FixedEmbeddingComposite with minorminer.find_clique_embedding, and set things by hand, but adds a level of complexity in the script.

alexzucca90 avatar Feb 04 '21 01:02 alexzucca90

You can also do SpinReversalTransfromComposite(DWaveCliqueSampler()), is SRTs on the logical problem good enough or do you need them on the embedded problem?

arcondello avatar Feb 04 '21 16:02 arcondello

I kinda wanted to have them on the embedded problem, but I'll test it first.

alexzucca90 avatar Feb 04 '21 17:02 alexzucca90

I don't love the switching behavior of not using the extended J range based on a seemingly unrelated parameter - we know the relation but the user probably wouldn't intuit it.

One solution would be to emit a warning, but I find that unsatisfying as well.

arcondello avatar Feb 04 '21 17:02 arcondello