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

feat(circuits): add QAOAPathCircuit class #14

Merged
merged 4 commits into from
Mar 26, 2024
Merged

feat(circuits): add QAOAPathCircuit class #14

merged 4 commits into from
Mar 26, 2024

Conversation

pedrorrivero
Copy link
Member

@pedrorrivero pedrorrivero commented Feb 24, 2024 •
edited
Loading

Summary

Adds new QAOAPathCircuit class to create parameterized QAOA quantum circuits for path acyclic graphs, at varying number of qubits and two-qubit depth.

Details and comments

  • This is a utility class to avoid building these (well-known) circuits manually in present and upcoming notebooks.
  • Needed for the Utility Scaling notebook, demoed at IBM Quantum Practitioner's Forum 2023.

@pedrorrivero pedrorrivero added feature New feature request PL-2 Priority level 2/5 → Medium-high labels Feb 24, 2024
@pedrorrivero pedrorrivero self-assigned this Feb 24, 2024
@coveralls
Copy link

coveralls commented Feb 24, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 8441529388

Details

  • 41 of 41 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%
Totals Coverage Status
Change from base Build 8441486427: 0.0%
Covered Lines: 60
Relevant Lines: 60
💛 - Coveralls

@pedrorrivero pedrorrivero changed the title feat(circuits): add QAOALineCircuit feat(circuits): add QAOALineCircuit class Feb 24, 2024
@pedrorrivero pedrorrivero changed the title feat(circuits): add QAOALineCircuit class feat(circuits): add QAOALineCircuit class Feb 24, 2024
@pedrorrivero pedrorrivero mentioned this pull request Feb 28, 2024
Merged
4 tasks
a-matsuo
a-matsuo reviewed Mar 18, 2024
View reviewed changes
Copy link
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.

I left some comments. It seems good. However, the name, 'QAOALineCircuit' might need to change.

quantum_enablement/circuits/_qaoa.py Outdated Show resolved Hide resolved
quantum_enablement/circuits/_qaoa.py Show resolved Hide resolved
test/unit/circuits/test_qaoa.py Outdated Show resolved Hide resolved
@pedrorrivero pedrorrivero changed the title feat(circuits): add QAOALineCircuit class feat(circuits): add QAOAPathCircuit class Mar 18, 2024
a-matsuo
a-matsuo previously approved these changes Mar 22, 2024
View reviewed changes
Copy link
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!

@pedrorrivero pedrorrivero merged commit 967e1c2 into main Mar 26, 2024
11 checks passed
@pedrorrivero pedrorrivero deleted the qaoa branch March 26, 2024 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@a-matsuo a-matsuo a-matsuo left review comments

@ritajitmajumdar1 ritajitmajumdar1 Awaiting requested review from ritajitmajumdar1

Assignees

@pedrorrivero pedrorrivero

Labels
feature New feature request PL-2 Priority level 2/5 → Medium-high
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@pedrorrivero @coveralls @a-matsuo