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.

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

Pennylane #61

Merged
merged 6 commits into from
Nov 23, 2023
Merged

Pennylane #61

merged 6 commits into from
Nov 23, 2023

Conversation

stroblme
Copy link
Member

This PR sets the max_worker value in the pennylane simulator to None to prevent allocating more processes than the one the evaluation is running in.

Depends on #60

Signed-off-by: lc3267 <melvin.strobl@kit.edu>
Signed-off-by: lc3267 <melvin.strobl@kit.edu>
@pep8speaks
Copy link

pep8speaks commented Nov 13, 2023

Hello @stroblme! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2023-11-23 19:51:43 UTC

@stroblme stroblme requested review from cDenius and removed request for cDenius November 13, 2023 15:11
Signed-off-by: lc3267 <melvin.strobl@kit.edu>
@stroblme stroblme mentioned this pull request Nov 13, 2023
@stroblme stroblme requested review from a team, eileen-kuehn and cDenius and removed request for a team November 13, 2023 15:28
Copy link
Member

@eileen-kuehn eileen-kuehn left a comment

Choose a reason for hiding this comment

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

When ignoring all the not-pennylane related stuff, it looks great! 🥳
Please go for it 👍

@stroblme stroblme merged commit aee15fb into main Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants