Appointat

Results 29 comments of Appointat

@camel-ai/camel-maintainers Hi, could you please review this PR? This PR has been proposed for several months now, and we've made many modifications; it should be close to merging. thanks.

> Hey @Appointat, thanks for your detailed polish on the docstring! If other parts is also good for you could you approve this PR? Also one quick tip, maybe it's...

@FUYICC Hi, is the pr still in progress? Let me know if you have any difficulties.

@Wendong-Fan Hi, could you please help me add `GROQ_API_KEY` to the GitHub secret (I will DM you the secrete key value)? Thanks a lot in advance.

@camel-ai/camel-maintainers @Wendong-Fan hi guys, this pr is fixed finally, could you please review it? thanks

@Wendong-Fan I have seen an error in pytest ``` FAILED test/models/test_groq_llama3_model.py::test_groq_llama3_model[ModelType.GROQ_LLAMA_3_8_B] - ValueError: Invalid `model_path` (meta-llama/Meta-Llama-3-8B-Instruct) is provided. Tokenizer loading failed. FAILED test/models/test_groq_llama3_model.py::test_groq_llama3_model[ModelType.GROQ_LLAMA_3_70_B] - ValueError: Invalid `model_path` (meta-llama/Meta-Llama-3-70B-Instruct) is provided....

Thank you, I will fix the conflits.

@Wendong-Fan I have checked the code, but I cannot review the code. I think it is ok to be merged. Thank you.

@Konisberg Thanks, the deductive_reasoner_agent is in the branch [feature/multi-agent-prod](https://github.com/camel-ai/camel/tree/feature/multi-agent-prod). Well, this is a by-product of the multi-agent project. At present, this part has not been merged into the main branch....

Hi @ZhangT-tech , do you think this PR still needs to be reviewed?