【代码贡献】修复并加速 QAOA 的 CVaR/Gibbs 风险目标函数 - #40
Open
zsszyx wants to merge 1 commit into
Open
Conversation
zsszyx
marked this pull request as ready for review
August 9, 2026 10:33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
参赛说明
问题
QAOA 的两个风险敏感目标函数存在正确性和数值稳定性问题:
1 / alpha归一化。calculate_energy每次调用都重复执行sympy.lambdify。修复内容
alpha和temperature严格校验(0, 1],防止零值、布尔值、NaN 和无穷值进入计算。验证
python -m pytest test/QAOA -q:24 passedpython -m pytest test -q:38 passed477.087 us降至0.161 us(约2966x,主要收益来自避免重复lambdify)。