Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

Fix for Incorrect translation of integer variables in ADMMOptimizer #1393

Merged
merged 1 commit into from
Oct 27, 2020

Conversation

adekusar-drl
Copy link
Contributor

Summary

This PR fixes #1392. An exception is thrown when trying to solve a problem with integer variables via ADMMOptimizer.

Details and comments

This PR Includes:

  • A fix for ADMMOptimizer.
  • A new test that now has integer variables.

@woodsp-ibm woodsp-ibm added Changelog: Bugfix Include in the Fixed section of the changelog stable backport potential The bug might be minimal and/or import enough to be port to stable labels Oct 27, 2020
@adekusar-drl
Copy link
Contributor Author

@woodsp-ibm If a backport is required, what should I do?

@woodsp-ibm
Copy link
Member

@adekusar-drl You do not need to do anything - its a label that would be used later when/if we come to do a bug fix release that such PRs could be cherry picked across a part of that stable bug fix release.

@adekusar-drl adekusar-drl merged commit 15d8c5f into qiskit-community:master Oct 27, 2020
manoelmarques pushed a commit to manoelmarques/qiskit-aqua that referenced this pull request Nov 9, 2020
@adekusar-drl adekusar-drl deleted the admm_integer branch November 12, 2020 14:01
manoelmarques pushed a commit to qiskit-community/qiskit-optimization that referenced this pull request Jan 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changelog: Bugfix Include in the Fixed section of the changelog stable backport potential The bug might be minimal and/or import enough to be port to stable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect translation of integer variables in ADMMOptimizer
3 participants