Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix bug in max-cut solver (issue #611) (backport #635) #641

Merged
merged 1 commit into from
Oct 25, 2024
Merged

Fix bug in max-cut solver (issue #611) (backport #635) #641

merged 1 commit into from
Oct 25, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Oct 25, 2024

This PR fixes the bug in the max-cut solver as reported in #611

This is an automatic backport of pull request #635 done by Mergify.

@TolisChal @mergify
Fix bug in max-cut solver (issue #611) (#635)
6e614c9 
* fix bug in max-cut solver (issue #611)

* fix copyrights

* add unknown-option-value as an error message to be ignored by pylint

* disable pylint too-many-positional-arguments error for the lines it pops up

* fix copyrights

* fix max-cut bug

(cherry picked from commit 8a7e09e)
@mergify mergify bot merged commit 34a8a05 into stable/0.6 Oct 25, 2024
17 checks passed
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11511216173

Details

  • 27 of 29 (93.1%) changed or added relevant lines in 17 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 92.857%
Changes Missing Coverage Covered Lines Changed/Added Lines %
qiskit_optimization/applications/max_cut.py 0 2 0.0%
Totals Coverage Status
Change from base Build 10312589374: 0.0%
Covered Lines: 4446
Relevant Lines: 4788
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t-imamichi t-imamichi t-imamichi approved these changes

@stefan-woerner stefan-woerner Awaiting requested review from stefan-woerner stefan-woerner is a code owner

@woodsp-ibm woodsp-ibm Awaiting requested review from woodsp-ibm woodsp-ibm is a code owner

Assignees
No one assigned
Labels
automerge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@coveralls @t-imamichi @TolisChal