Skip to content

Add indicator constraint support in from_docplex_mp translator#163

Merged
t-imamichi merged 5 commits intoqiskit-community:mainfrom
t-imamichi:indicator
Jun 17, 2021
Merged

Add indicator constraint support in from_docplex_mp translator#163
t-imamichi merged 5 commits intoqiskit-community:mainfrom
t-imamichi:indicator

Conversation

@t-imamichi
Copy link
Copy Markdown
Collaborator

@t-imamichi t-imamichi commented Jun 15, 2021

Summary

Adds support of indicator constraints in from_docplex_mp.
This implementation is based on #151.

Fixes #13
Closes #151

Details and comments

@t-imamichi t-imamichi added the Changelog: New feature Include in the Added section of the changelog label Jun 15, 2021
@t-imamichi t-imamichi requested a review from a-matsuo June 15, 2021 14:01
@t-imamichi t-imamichi marked this pull request as draft June 15, 2021 14:05
@t-imamichi t-imamichi force-pushed the indicator branch 3 times, most recently from 07f5b81 to 139ca3a Compare June 15, 2021 15:57
Co-authored-by: Atsushi Matsuo <MATSUOA@jp.ibm.com>
@t-imamichi t-imamichi marked this pull request as ready for review June 15, 2021 15:58
Comment thread qiskit_optimization/translators/docplex_mp.py Outdated
Comment thread qiskit_optimization/translators/docplex_mp.py Outdated
woodsp-ibm
woodsp-ibm previously approved these changes Jun 15, 2021
a-matsuo
a-matsuo previously approved these changes Jun 16, 2021
Copy link
Copy Markdown
Contributor

@a-matsuo a-matsuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@t-imamichi
Copy link
Copy Markdown
Collaborator Author

I merged #170 and resolved conflicts

@t-imamichi t-imamichi merged commit b0b706e into qiskit-community:main Jun 17, 2021
@t-imamichi t-imamichi deleted the indicator branch June 17, 2021 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: New feature Include in the Added section of the changelog priority: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for docplex.mp.constr.IndicatorConstraint

3 participants