dwavesystems / dwavesystems/penaltymodel

Separate Specification from PenaltyModel Class

Open
#78 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
20
Forks
22
PR merge metrics
No merged PRs in 30d

Description

**Current Problem**
Currently, the Specification class is a super class of PenaltyModel. This is a bit confusing as Specification objects get passed to penaltymodel-mip, -maxgap, and -lp in order to generate a PenaltyModel object.

This also leads to issues between `classical_gap` and `min_classical_gap` in PR #74. (Most notably that the subclass `classical_gap` arg gets passed to the super's `min_classical_gap`).

**Proposed Solution**
Refactor the code so that PenaltyModel is a subclass of BQM

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.