dincli: `task model-owner register-request --manifestCID` fails because the registry contract isn't initialised
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 79/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- python, solidity
- 領域
- blockchain, cli
調査の方向性
dincli/cli/task.pyを読み、register_requestとupdate-manifest-requestを比較してください。レジストリコントラクトの初期化を移動し、--manifestCID経由とマニフェストファイル経由のどちらでも手数料の取得を使えるようにしてから、CID経由のテストを追加してください。register-request --manifestCID <cid>がUnboundLocalErrorを発生させずにトランザクション送信まで到達すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Found by @Santiagocetran while reviewing #291 and #294, and confirmed in the source. Not yet reproduced on a live network.
In dincli/cli/task.py, register_request only sets dinregistry_contract = ctx.obj.get_deployed_din_registry_contract() inside the if not manifestCID: block (the manifest-file path). When a user passes --manifestCID, that block is skipped, and the later fee lookup (dinregistry_contract.functions.openSourceFee() / proprietaryFee()) fails with an UnboundLocalError before any transaction is sent.
Expected: register-request --manifestCID <cid> works the same as the manifest-file route.
Likely fix: move the get_deployed_din_registry_contract() call above the if not manifestCID: block, as update-manifest-request already does, and add a test for the CID path.
Until it's fixed, the docs should only show the --manifestpath route (see #294).
- 主要言語
- Python
- スター
- 4
- フォーク
- 7
- 平均マージ
- 3日 4時間
- マージ済み PR（30日）
- 34
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
InfiniteZeroFoundation/DevNet のほかの issue
-
ROADMAP.md: refresh "What is still open for P3" (several items were fixed in October)対応中かも @umermjd11 が 2 日前に担当しました。 オープンdocumentation good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
InfiniteZeroFoundation/DevNet#285 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Update the 25 translated organisation READMEs to match the English version対応中かも @mrCabez が今日担当しました。 オープンgood first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
InfiniteZeroFoundation/DevNet#277 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Update the client-service guide to explain training and uploading separately対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
InfiniteZeroFoundation/DevNet#266 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
InfiniteZeroFoundation/DevNet#299 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Long run
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
InfiniteZeroFoundation/DevNet#298 ·
メンテナーはふだん 1 日以内に返信
InfiniteZeroFoundation/DevNet の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
-
Validate finite output clamping limits in StepPredictor.prepare_clamping_params to prevent silent NaN/Inf propagation対応中かも @muskan-creates352 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
mllam/neural-lam#848 ·
メンテナーはふだん 2 日以内に返信
-
infra
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
fab-geocommuns/RNB-coeur#1166 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
deepspeedai/DeepSpeed#8951 ·
メンテナーはふだん 2 日以内に返信