Instructors and TAs need a way to test an assignment before releasing it to students
まだ誰も着手していません。
評価
調査の方向性
Start by reading code_editor.py's run_code and _verify_student, then compare submission.py's upload_submission behavior. Decide how staff testing should work for upload-only assignments, and inspect reviewGrades and manageSubmissions for roster handling. Done means staff can test before release without test data appearing as student submissions or affecting grades and counts.
索引モデルが issue の本文から書いたものです。
説明
Currently, both the code editor (run_code/submit_code in code_editor.py) and file-upload submission (upload_submission in submission.py) block non-student callers — _verify_student has no staff bypass, and upload_submission has an explicit comment: "Instructors/TAs should not upload submissions on behalf of students." There's no way for staff to verify an assignment's autograder works before publishing it to students.
run_code is already ephemeral (no Submission row is written — "Run code without creating a submission" per its own docstring), which makes it the natural place to allow staff through for code-editor assignments. Two things stand in the way even there:
_verify_studentincode_editor.pyhas no staff bypass.run_codeexplicitly rejects unpublished assignments (if not assignment.published: raise BadRequestError(...)) — but testing before publishing is the entire point of this request, so that gate needs a staff exception too.
Upload-only assignments (enable_code_editor=False) have no ephemeral equivalent — upload_submission always persists a Submission row, and there's currently no flag on Submission to distinguish a staff test run from a real one. Supporting staff testing there means either adding a Submission flag (schema migration) or a separate non-persisted "dry run" execution path, and either way, reviewGrades/manageSubmissions' rosters and any grade-count/dashboard logic would need to exclude staff test data — which they don't do today for any role (see the linked roster-filtering issue).
Reported by Cecilia (yczhangcc) during review of PR #355.
- 主要言語
- Python
- スター
- 13
- フォーク
- 0
- 平均マージ
- 2時間 4分
- マージ済み PR（30日）
- 1
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
kiat/codeAssist のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
kiat/codeAssist#391 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
kiat/codeAssist#364 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
kiat/codeAssist#393 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 63/100
kiat/codeAssist#392 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
kiat/codeAssist#390 ·
kiat/codeAssist の issue をすべて見る
似ている issue
-
vis.image() reports "Unsupported number of image channels: 224" for an HxWxC image対応中かも @piyush182004 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
fossasia/visdom#1997 · コメント 3 件 ·
メンテナーはふだん 6 日以内に返信
-
Byse new domainオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
Gujal00/ResolveURL#1525 ·
メンテナーはふだん 1 日以内に返信
-
Serialisation fails with pydantic 2.14: wrap serialisers pass SerializationInfo as index_key対応中かも @hugohills-regnosys が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
finos/rune-python-runtime#45 ·
-
[Bug]: LangGraph Store sync put bypasses namespace validation対応中かも @ktz03 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
volcengine/OpenViking#6050 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
hummingbot/condor#283 ·
メンテナーはふだん 1 日以内に返信