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

PauliOperatorの掛け算に関するバグ #210

Merged
merged 13 commits into from
Feb 2, 2022
Merged

Conversation

kotamanegi
Copy link
Collaborator

@kotamanegi kotamanegi commented Jan 21, 2022

close #209

PR #185 からPauliOperatorの掛け算に関するバグ修正コミットを抜き出したものです。

主にチェックしたい点

  • 本当にコードが正しいか (今回修正していない点も確認していただきたいです)
  • できればテストも追加したい (手が空いている人にお願いします)

@kotamanegi kotamanegi marked this pull request as ready for review January 21, 2022 11:47
@Hiroya-W
Copy link
Contributor

本当にコードが正しいか確認するためにも、早めにテストコードを追加した方が良さそうですね

@Hiroya-W
Copy link
Contributor

#pragma omp parallel forの使い方が良くなく、バグになっているので、対応する必要がありそうです

@kosukemtr
Copy link
Collaborator

PR #185 でも openmp は対応していました。ありがとうございます。

Copy link
Contributor

@KowerKoint KowerKoint left a comment

Choose a reason for hiding this comment

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

LGTMです

Copy link
Contributor

@Hiroya-W Hiroya-W left a comment

Choose a reason for hiding this comment

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

用意しておきたいテストはこれで十分だと思います。
LGTMです!

@kotamanegi kotamanegi merged commit d14cbb7 into dev Feb 2, 2022
@kotamanegi kotamanegi deleted the 209-multiply-pauli branch February 2, 2022 00:05
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.

PauliOperator * PauliOperatorの処理が間違っている
5 participants