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

interallでの演算子の入れ替えに関するバグ #6

Merged
merged 1 commit into from May 24, 2021
Merged

interallでの演算子の入れ替えに関するバグ #6

merged 1 commit into from May 24, 2021

Conversation

mitsuaki1987
Copy link
Member

スタンダードモードでInterallのファイルが生成される場合にm
c1+ c2 c3+ c4 = c3+ c4 c1+ c2 = - c1+ c4 c3+ c2 = - c3+ c2 c1+ c4
という演算子の入れ替えを許して相互作用を合算していたが、
沿え字が1=2, 3=4, 1=4, または3=2 の状況を考慮しておらず正しくない合算をしていた。(反交換のお釣りの項がないため)
このバグはS>1/2かつ自己相互作用(D項もしくは極端に短い周期)の場合に間違った解を生じる。

この修正では1=2, 3=4, 1=4, または3=2 には演算子の合算は行わないようにした。

The general interaction Uinterall) was merged as
c1+ c2 c3+ c4 = c3+ c4 c1+ c2 = - c1+ c4 c3+ c2 = - c3+ c2 c1+ c4
But, the cases that 1=2, 3=4, 1=4, and 3=2 were not considered.
This bug caused incorrect result for e.g. S>1/2 && self interaction (D-term or very short periodicity).
@k-yoshimi k-yoshimi merged commit fb4b61a into issp-center-dev:master May 24, 2021
@k-yoshimi
Copy link
Contributor

@mitsuaki1987
修正ありがとうございます。mergeしました。

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.

None yet

2 participants