Skip to content

Circuit processing#133

Merged
Elscrux merged 13 commits into
developfrom
circuit-processing
Jun 25, 2026
Merged

Circuit processing#133
Elscrux merged 13 commits into
developfrom
circuit-processing

Conversation

@ekulos-code

Copy link
Copy Markdown
Contributor

Add circuit execution and optimization

@ekulos-code ekulos-code requested a review from koalamitice March 10, 2025 13:18

@koalamitice koalamitice left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Most of the code looks good! I made some comments. Feel free to ask questions if I did not explain them well enough :)

Comment thread solvers/circuitprocessing/projectq-executor/projectq_executor.py Outdated
@koalamitice

Copy link
Copy Markdown
Member

General comment: The test cases in this PR currently fail because of Python Dependency issues with existing solvers. This PR therefore depends on the finalization of #132 , which fixes this issues by adding a virtual environemnt for every Python solver.

@koalamitice koalamitice added the type: enhancement New feature or request label Mar 12, 2025
@sonarqubecloud

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

@ekulos-code ekulos-code force-pushed the circuit-processing branch from 0ca1d22 to c461dfe Compare May 7, 2026 16:52
@Elscrux Elscrux self-requested a review June 1, 2026 12:18

@Elscrux Elscrux left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This looks really polished already, great job! Please let me know what do you think are the things that are not really clear right now.

Comment thread solvers/circuitprocessing/circuitexecution/requirements.txt Outdated
Comment thread solvers/circuitprocessing/circuitoptimizing/requirements.txt Outdated
@sonarqubecloud

sonarqubecloud Bot commented Jun 5, 2026

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

@Elscrux

Elscrux commented Jun 21, 2026

Copy link
Copy Markdown
Member

Thanks for resolving everything, to me this looks pretty good now. The remaining steps to take seem to be:

@Elscrux Elscrux merged commit b1a82a2 into develop Jun 25, 2026
4 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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

Labels

type: enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants