Alexander Condello

Results 257 comments of Alexander Condello

I'll just add that the new default `chain_strength` added in [dwave-system 0.9.13](https://github.com/dwavesystems/dwave-system/releases/tag/0.9.13) will automatically scale with your problem.

Would be pretty straightforward to add another header line, ``` # vartype=SPIN # offset=42 0 1 1.000000 0 2 1.000000 0 3 1.000000 1 2 1.000000 1 3 1.000000 2...

~This implementation has some issues, specifically that `set_quadratic` etc do not respect the vartype of the BQM~ (edit: I see the `._bqm` property now). The existing [.SpinView](https://github.com/dwavesystems/dimod/blob/e6b0ea7e8bf614bc532bb85337aeb0060215fc78/dimod/core/bqm.py#L1377) and [.BinaryView](https://github.com/dwavesystems/dimod/blob/e6b0ea7e8bf614bc532bb85337aeb0060215fc78/dimod/core/bqm.py#L1312) are...

We could combine the `EitherView` with the existing `VartypeView` and use that instead of the two subclasses, which would make `change_vartype` well defined with fewer overall layers of abstraction.

For posterity, this can (as of 0.9.11) be done simply with ``` sampleset.record.energy = bqm.energies(sampleset) ``` It still might be worth implementing a method, but the one-liner isn't too bad.

Hi @yurivict , there [are instructions](https://github.com/dwavesystems/minorminer#installation-2) for using `cmake` to run the tests. Otherwise this is intended to be used a header only library.

Worth noting that `networkx.Graph` is not itself JSON-serializable, Bokeh has implemented a custom encoder `BokehJSONEncoder`. So I think the request is really "Graph output should be serializable by Bokeh's custom...

IMO this is not an easy fix. Simplest would be `del G.vartype`.

Ah, got it, comes from https://github.com/dwavesystems/dimod/blob/126a9056c4226d6955395620efb94ebee07581a5/dimod/converters.py#L49 Ok. I think your workaround it best for now.

I believe changing line 4 to `p cnf _ _` will fix it. But the function should raise a better exception.