Radomir Stevanovic
Radomir Stevanovic
Thank you @29rou, these are great finds. We'll look into them. Unless you'd like to? Great first step would be enabling universal builds for macos, similar to how cibuildwheel does...
Done in https://github.com/dwavesystems/dimod/pull/1290.
FWIW, I like @pau557's suggestion. The next logical step is to generalize it to store one polynomial term/coefficient per line: ``` 42 a 1.0 a b 2.0 b c 3.0...
I prefer categorical option to boolean because it allows future expansion. Accepting a callable (in addition) is trivial and provides an absolute flexibility.
IMO, "list" should include fields from all samplesets. We might consider splitting "list" into "list-all" and "list-existing", although "squeeze" is conceptually already very similar to "list-existing". @arcondello, thoughts?
I don't see how `numpy.squeeze` is relevant here, but I agree it's too general. `unique` sounds much better.
@boothby, would you prefer we keep shaving that yak (from 6dcee69702184526345246bcdc9644f807bacb47, [test_lib.py#L855-L884](https://github.com/dwavesystems/minorminer/blob/6dcee69702184526345246bcdc9644f807bacb47/tests/test_lib.py#L855-L884)), exclude osx-3.9, or remove kbd int tests altogether?
To unblock the release, I patched these tests to skip on osx-py39 as well. That means we're now testing for keyboard interruptibility only on Linux and macOS py35-py37. This issue...
An alternative to automated fallback is adding a possibility of "manual override": to e.g. prefer `penaltymodel-maxgap`.
Default "extra" would solve this problem. Unfortunately, it's [far from being available](https://github.com/pypa/setuptools/issues/1139).