kubeedge/ianvs

Inconsistent with interface name 'initial_model_url' and environment variable 'base_model_url'

オープン

#14 opened on 2022/08/09

GitHub で見るコントリビューターガイド
 (5 件のコメント) (0 件のリアクション) (1 人の担当者)Python (168 件のフォーク)auto 404
good first issuekind/cleanup

Repository metrics

Stars
 (178 個のスター)
PR merge metrics
 (PR metrics pending)

説明

What would you like to be added/modified: While configuring algorithm.yaml, the pre-trained model url is specified as initial_model_url, however in test_algorithms/basemodel.py it is parsed and renamed as base_model_url.

Why is this needed: This inconsistency might be confusing for algorithm developers. May I possibly recommend revising these two names to the same one rather than keeping this inconsistency.

コントリビューターガイド

調査方針
リポジトリ内の `initial model url` と `base model url` のすべての出現箇所を検索してください。プロジェクトがどちらの名前を好むかを判断し、それに応じてコード、テスト、ドキュメントを更新してください。`algorithm.yaml` の設定と `test algorithms/basemodel.py` で文脈を確認してください。
技術スタック
python
領域
backend
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
GitPython
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
80