garylin2099

> Are the test cases enough? Not enough yet. Since its target is not main, I suggest merging it first if needed. Tests must be supplemented if merge is filed...

Let me clarify the question, so DI is expected to generate a string for your downstream tasks, and you want the string to contain the file name instead of the...

You are right, the problem is that the llm gave an empty response. Your running record:  What it is supposed to be (from my run just now):  May...

Can you try using the original one below to see if the issue still exists? You have access to openai in github codespace without proxy. ``` OPENAI_API_BASE: "https://api.openai.com/v1" ``` >...

I recommend using main, too. From your log, the original temperature issue is fixed, which is good. The problem is Gemini did not follow the instruction to generate response in...

先合入，后面优化

 https://github.com/geekan/MetaGPT/actions/runs/7726960897/job/21064351903?pr=736 Reached test coverage 86%. No lower than target branch dev. In terms of 8 test failures, 6 of them depend on...

 https://github.com/geekan/MetaGPT/actions/runs/7735868125/job/21092824632 Test coverage at 87%, higher than target branch (dev), all tests passed (5 failures at dev not related to this PR)

> 1. Don't write comments that don't make sense, if the subclass's methods are nothing special, it should probably use the parent's comments > 2. Notice that some logic has...

fixed in https://github.com/geekan/MetaGPT/pull/717