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

parameter p doesn't seem like it is used in the get_qaoa_labs_objective function #58

Open
shreehari117 opened this issue May 6, 2024 · 1 comment

Comments

@shreehari117
Copy link

The get_qaoa_labs_objective function takes N and p, along with other parameters. The get_qaoa_labs_objective calls get_qaoa_objective, where p does not seem like it is used.

@jhnnsjkbk
Copy link

Hey! I stumbled across the same thing. p is never used. However, p is indirectly set by the length of e.g., theta such as here:

p = int(len(theta) / 2)
. It's quite uncommon and looks a bit hacky to me, but I might miss something. Hope this helps. Essentially, you would want to not set p but the length of e.g. theta.

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

No branches or pull requests

2 participants