`for` loops only accept sets and ranges — not `bit[n]`, arrays, aliases or index expressions
メンテナーはふだん 1 日以内に返信
評価
調査の方向性
Start in Qasm3Visitor._visit_forin_loop at src/pyqasm/visitor.py (around line 2462), where the type check on statement.set_declaration rejects everything except DiscreteSet and RangeDefinition. Move that branch into a helper that returns a list of values, then add the array case first, since it doesn't depend on other work. Done when tests/qasm3/test_loop.py covers each iterable form and an index expression, a write to the loop variable leaves the source array unchanged, and a multi-dimensional array raises ValidationError.
索引モデルが issue の本文から書いたものです。
説明
Limitation
For loops states that the iterated <values> may
be any of:
- a discrete set, e.g.
{1, 2, 3} - a range expression, e.g.
[0:2:10] - a value of type
bit[n], or the target of aletstatement aliasing classical bits - a value of type
array[<scalar>, n](one-dimensional)
and adds: "It is valid to use an indexing expression (e.g. my_array[1:3]) to arrive at one of
the types given above."
pyqasm implements only the first two. Any identifier or index expression as the iterable is
rejected.
Example QASM failure
Iterating a bit register:
OPENQASM 3.0;
bit[4] b = "1010";
for bit x in b { }
// ValidationError: Unexpected type <class 'openqasm3.ast.Identifier'> of set_declaration in loop.
Iterating an array:
OPENQASM 3.0;
array[int[8], 3] a = {1, 2, 3};
for int[8] i in a { }
// ValidationError: Unexpected type <class 'openqasm3.ast.Identifier'> of set_declaration in loop.
Iterating an index expression:
OPENQASM 3.0;
array[int[8], 4] a = {1, 2, 3, 4};
for int[8] i in a[1:2] { }
// ValidationError: Unexpected type <class 'openqasm3.ast.IndexExpression'> of set_declaration in loop.
Iterating a classical alias:
OPENQASM 3.0;
bit[4] b;
let c = b;
for bit x in c { }
// ValidationError: Qubit register b not found for aliasing
Change Requested
Extend the loop's iterable resolution to accept, in addition to the existing set and range
forms:
- An
Identifiernaming abit[n]— loop variable typebit, iterating index0..n-1. - An
Identifiernaming a one-dimensionalarray[<scalar>, n]— loop variable type
<scalar>, promoted to the declared loop variable type. - An
Identifiernaming a classical alias. - An
IndexExpressionthat evaluates to any of the above.
Two spec rules to enforce while doing so:
- Iteration order is guaranteed sequential by index:
iden[0], theniden[1], and so on. - Assigning to the loop variable inside the body must not modify the underlying array element.
A multi-dimensional array as the iterable must raise a ValidationError — the spec restricts
this to one-dimensional arrays.
Implementation Details
- The rejection is in
Qasm3Visitor._visit_forin_loopatsrc/pyqasm/visitor.py:2462. It
branches on the concrete type ofstatement.set_declarationand falls through for anything
that is not aDiscreteSetorRangeDefinition. - Factor the branch into a helper that returns a plain Python list of values to iterate. That
isolates the new cases and keeps the unrolling loop below unchanged. - Since the loop body is unrolled per value, the "assignment to the loop variable does not
write back" rule falls out naturally from binding a copy into the loop scope each
iteration — but add an explicit test, since it is easy to regress with a by-reference bind. - Depends on the
bit[n]representation work for case 1 and on the classical-alias work for
case 3; case 2 (arrays) is independent and could land first. - Tests:
tests/qasm3/test_loop.py— one case per iterable form, an index-expression iterable,
a write to the loop variable asserting the source array is unchanged, and a
multi-dimensional array asserting a cleanValidationError.
- 主要言語
- Python
- スター
- 29
- フォーク
- 33
- 平均マージ
- 2日 4時間
- マージ済み PR(30日)
- 15
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
qBraid/pyqasm のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 63/100
メンテナーはふだん 1 日以内に返信
-
enhancement error-handling program-validation
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
Move the QASM 2 printer and its branch pre-processing out of qasm2.py対応中かも @danielgaskins が 9 日前に担当しました。 オープンenhancement qasm2
難易度 4/5 3〜5日 初心者へのやさしさ 22/100
qBraid/pyqasm#447 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
[BUG] dumps() emits extern calls without a semicolon, producing unparsable QASM対応中かも @Jah-yee が 17 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 25/100
qBraid/pyqasm#438 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Parse failures raise ValidationError with an empty message; the ANTLR diagnostic is lost対応中かも @tonycoder-hub が 11 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 22/100
qBraid/pyqasm#436 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
foxglove/mcap#1907 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
namespace operations
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
EclipseFdn/open-vsx.org#14623 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
developmentseed/titiler-cmr#261 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
andrewyng/openworker#774 ·
メンテナーはふだん 1 日以内に返信
-
aggfuncオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
worldbank/blackmarblepy#160 ·