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

Feature/update test #65

Merged
merged 5 commits into from
Sep 9, 2024
Merged

Feature/update test #65

merged 5 commits into from
Sep 9, 2024

Conversation

Kotani-UT
Copy link
Collaborator

古典ビットでの制御に関するコードとテストの追加をしました。
qiskit_algorithms依存のテストの削除とu1,u2,u3ゲートのサポートの削除をしました。

@kkyusuke kkyusuke self-requested a review September 7, 2024 01:26
@kkyusuke
Copy link
Collaborator

kkyusuke commented Sep 7, 2024

[deb0648]でqiskit_algorithmsを使用しているテストコードが削られていますが、これはなぜなのでしょうか?

@Kotani-UT
Copy link
Collaborator Author

・qiskit_algorithmのサポートが終了している
・qiskit_algorithmがu2を利用している
の2点からqiskit_algorithmsを使用しているテストを削除しました。
結局u1,u2,u3のサポートを残したままでよかったのでしょうか?

またvqe,qaoaといった回路に関しては、
https://learning.quantum.ibm.com/tutorial/variational-quantum-eigensolver
https://learning.quantum.ibm.com/tutorial/quantum-approximate-optimization-algorithm
にqiskit1.0.2に対応したチュートリアルがあるのですが、これらを利用して追加したほうが良いでしょうか?

@kkyusuke
Copy link
Collaborator

kkyusuke commented Sep 9, 2024

そういえば、そういう経緯でしたね。
それでは、このPRは問題なさそうなのでマージすることにして、次のPRでは提示していただいたURLに沿ったテストを作成いただく、という流れはいかがでしょうか?

Copy link
Collaborator

@kkyusuke kkyusuke left a comment

Choose a reason for hiding this comment

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

LGTM

@kkyusuke
Copy link
Collaborator

kkyusuke commented Sep 9, 2024

一部のテスト作成は次のPRに回し、マージします

@kkyusuke kkyusuke merged commit 46be0f6 into develop Sep 9, 2024
5 checks passed
@kkyusuke kkyusuke deleted the feature/update-test branch September 9, 2024 06:34
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.

2 participants