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

modify latex.py to use circuit parser #51

Merged
merged 10 commits into from
Jan 27, 2022
Merged

modify latex.py to use circuit parser #51

merged 10 commits into from
Jan 27, 2022

Conversation

kawamochi
Copy link
Contributor

parserを使うように変更した

@kawamochi
Copy link
Contributor Author

現状textのexample2とexample4で壊れることを確認

SWAPの方はLatexの書き方調べたら行けそう

example2の方はDense Matrixがなんかでかい(?)インデックスの指定間違えてそう

@kawamochi
Copy link
Contributor Author

Multi gateでtarget indexが連続でない場合に対応

swapに対応

時間ギリギリになったのでバグがあるか確認できていない

@kawamochi kawamochi marked this pull request as ready for review January 6, 2022 11:55
@Hiroya-W
Copy link
Contributor

Hiroya-W commented Jan 7, 2022

手元で確認してみていたんですが、テストケースとして用意した回路に関してはLaTeXでビルドして画像生成ができているようでした!
このままテストケース追加してマージまでしたいですね。

@Hiroya-W
Copy link
Contributor

Hiroya-W commented Jan 7, 2022

テストデータを使って、画像を自動で生成出来るようにスクリプトも用意した方が良さそう。これはこの後PRで対応しようと思います。
手元で環境作って画像生成させていたけど、ある方が便利だと思うし。

@Hiroya-W
Copy link
Contributor

LaTeX用のtexソースファイルの生成、LaTeXを使って画像を生成し、一致するか確かめるテストを追加しました。
生成されているものが予想通りのものか確認をしてほしいです。

@Hiroya-W
Copy link
Contributor

@kawamochi ↑お願いします

@kawamochi
Copy link
Contributor Author

OKです!
マージします

@kawamochi kawamochi merged commit dc25573 into main Jan 27, 2022
@Hiroya-W Hiroya-W deleted the 38-fix-latex-drawer branch March 11, 2022 06:03
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.

Latex Drawerの修正 生成された量子回路(LaTeX)に対するテスト書く
2 participants