Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improved parameters of SPSA #765

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

lockwo
Copy link
Contributor

@lockwo lockwo commented Apr 23, 2023

I was encountering some poor performance and was able to track down the cause to 2 places. 1) there was a mistake in the scaling of the new learning rate (it should be 10%, not 1% of total iterations), and 2) a better default learning rate is 0.05 * ((maxiter * 0.1) + 1) ** alpha (per https://docs.pennylane.ai/en/stable/_modules/pennylane/optimize/spsa.html#SPSAOptimizer), which for default values is actually 0.05 * ((200 * 0.1) + 1) ** 0.602 = 0.31.

@mhucka mhucka added the kind/bug-report Something doesn't seem to work label Dec 2, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
kind/bug-report Something doesn't seem to work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants