There is a bug in the break_cycle logic in CodeReview
まだ誰も着手していません。
評価
調査の方向性
composed_phase.py を開いて CodeReview.break_cycle を調査し、特に CodeReviewComment 中に phase_env がどのように設定されるかを確認してください。「 Finished」という conclusion が含まれるフィールドを確認し、サイクルの切断が review_comments を読み取ることを確認してください。ロジックが issue で特定されたフィールドをチェックし、周辺の workflow が引き続き期待どおりに動作すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
composed_phase.py
class CodeReview(ComposedPhase):
def break_cycle(self, phase_env) -> bool:
if "<INFO> Finished".lower() in phase_env['modification_conclusion'].lower():
return True
else:
return False
According to the prompt, " Finished" is only returned during the CodeReviewComment stage, and the conclusion of this stage will be stored in chat_env.env_dict['review_comments']. There seems to be a bug in the current code logic. I believe the correct code should be as follows:
class CodeReview(ComposedPhase):
def break_cycle(self, phase_env) -> bool:
# 修复bug，这个标识来自于review_comments
if "<INFO> Finished".lower() in phase_env['review_comments'].lower():
return True
else:
return False
- 主要言語
- Python
- スター
- 34.5k
- フォーク
- 4.3k
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
OpenBMB/ChatDev のほかの issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 76/100
-
CodeReviewModification never shows the programmer the six review regulations, so fixes are made blind to the checklist they are re-reviewed against対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 1/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
-
FunctionManager silently drops tool modules containing dataclasses with postponed annotations対応中かも @inchang-ing が 16 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 72/100
OpenBMB/ChatDev の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
aws-samples/sample-per-model-bedrock#243 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
FuRongJun-1999/dsh-memory#404 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
bug frontend
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
PennyLaneAI/catalyst#3337 ·
メンテナーはふだん 1 日以内に返信
-
Workflow.schedule stops when a cron deadline passes before sleep対応中かも @HuaTNA が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信