New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Penalty model lp #64
Penalty model lp #64
Conversation
4ef562c
to
f08baa9
Compare
Note: there are a couple packages in penaltymodel-lp's |
…ltymodel entrypoint for toy
…dditional valid state to make a proper or_gate
…y work for spin values, so replace bool-and with min(a,b)
…Add xor_gate test
… generate_bqm(..)
… that unspecified states are based on gap + lowest-specified-energy
…an qubo; and deal with single tuple error. Add some energy range tests
0594a64
to
84e54ca
Compare
…ments Also need to keep the dimod versions consistent accross the different requirements files
Also the file |
setup( | ||
name="penaltymodel-lp", | ||
install_requires=install_requires, | ||
entry_points={ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to add the packages
setup(
name="penaltymodel-lp",
install_requires=install_requires,
packages=packages,
entry_points={
FACTORY_ENTRYPOINT: ['lp = penaltymodel.lp:get_penalty_model']
}
)
In general this needs to have additional parameters added. I would use
from __future__ import absolute_import |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
except python_requires
should be
python_requires = '>=2.7,!=3.0.*,!=3.1.*,!=3.2.*,!=3.3.*'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. I'll push the change with the packages
first to make sure it fixes the CI errors. If that passes, I'll switch over and follow mip's setup.py
.
…so it's not penaltymodel's job to test this. Penaltymodel only deals in ising.
Closes #10 |
Please just review; do not merge.
Unit tests,
test_generation.py
andtest_interface.py
are still a work in progress. (read: please don't review them as they will change).PR will add linear programming as a penalty model.