Qulacs-Osaka / qulacs-osaka

Development branch of qulacs at Osaka Univ

MIT License

13 stars 6 forks source link

Comment encode fix #304

Closed WATLE closed 2 years ago

WATLE commented 2 years ago

247

読めないコメントを、http://docs.qulacs.org/ja/latest/index.html　とか、元がgate_matrixからのコピペだったりするものがあったので、補いました。ついでに　merge_circuit　で二つの回路のqubitが違っていても、定義上何の問題もないはずなので、エラー吐かないようにしました

WATLE commented 2 years ago

途中でrefactoring cppsim by KowerKoint #301　をマージしたので変更量がすごいことになってる...

WATLE commented 2 years ago

「merge_circuit　で二つの回路のqubitが違っていても、定義上何の問題もない」よく考えるとマージ元の回路のbitよりマージ先が少ないと少ないbitの回路に大きいbitのゲート入れようとしてエラー吐く

吐くけど、エラーをビットの数が違う時点で吐くか、　少し待ってエラーを吐くかの違いだし、　それでエラーにならないなら操作を許容したほうがよさそう

WATLE commented 2 years ago

コメント追加しました

WATLE commented 2 years ago

306 これもやった

forest1040 commented 2 years ago

merge_circuitは、qubit_countが一緒じゃないとダメなんですね。。そのうち必要であれば、拡張する便利メソッドとか用意するとよいかも知れませんね。。

LGTMです！