Radomir Stevanovic comments

Results 125 comments of


                                            Radomir Stevanovic

Inherit beta_range from sampler.parameters to sample()

This is in spirit of dwave-hybrid's runnables' behavior where `.run()` params override params set during block construction, but are overridden themselves with input state values.

SimulatedAnnealingSampler.sample method return type incorrectly documented

Fixed.

Potential dependency conflicts between dwave-neal and numpy

Thank you for bringing this to our attention, @NeolithEra. Due to existing dependency on numpy (both in Python and C/Cython code), I'm hesitant to remove the upper bound (ad 1),...

Potential dependency conflicts between dwave-neal and numpy

Obsoleted by move to dwave-samplers.

Sporadic test failure in appveyor 3.6 64 bit

Check [here](https://github.com/dwavesystems/dwave-neal/commit/838f096d3a601c66f1e7c0d282145bf9512fcc2f) and [there](https://github.com/dwavesystems/dwave-hybrid/commit/6ad26ea0d1edeff1d36ed6cd432955dcb36a4f5f).

Sporadic test failure in appveyor 3.6 64 bit

Or switch to CircleCI. :smile:

Sporadic test failure in appveyor 3.6 64 bit

Fixed with #99 (drop appveyor) and #103 (drop py36).

Allow solver as an argument to chimera_graph and pegasus_graph

No objections to the feature request, just a minor comment: `eq` operator is implied, so the filters can be simplified to `topology__type="..."`. If you aimed for explicit, ignore my comment.

Convert to namespace package

@arcondello, I like option (3). Seems simplest (with parameterized) / no new tests needed, covers everything, and can be easily removed when we kill the old ns.

Convert to namespace package

Also, we have quite a few tests, test files, and import statements like ``` from dwave_networkx.algorithms.canonicalization import rooted_tile ``` not just a simple `import dwave.plugins.networkx as dnx`. And given python's...