ChenXu233
commented
2 weeks ago 深入阅读代码后我还有个疑问：
这俩段代码相似度很高，可以考虑一下整一个函数出来进行代码复用？ 比如可以这样写
async def api_request(url, headers, body, timeout):
async with httpx.AsyncClient() as client:
try:
result = await client.post(url, headers=headers, json=body, timeout=timeout)
except httpx.ReadTimeout as e:
logger.critical(f"访问大模型超时, {e}")
result = None
except Exception as e:
logger.critical(r.text + str(e))
raise e
return result
class GLMModel(BaseLLMModel):
...
async def ask_model(
self,
question: str,
system_prompt: str = None,
message_history: list = (),
temperature: float = 0.01
):
"""向 ChatGLM 提问
- question：用户问题
- system_prompt：系统级提示词
- message_history: 消息历史列表
"""
url = self.get_api_url()
headers = self.get_headers()
body = self.get_body_template(temperature)
if message_history:
body["messages"] = message_history
if system_prompt:
sys_msg = {"role": "system", "content": system_prompt}
body["messages"].insert(0, sys_msg)
user_message = {"role": "user", "content": question}
body["messages"].append(user_message)
if result := api_request(url, headers, body, self.timeout):
#这下面可以依据不同的更改不同的逻辑
ans = result.json()["choices"][0]["message"]["content"]
return ans没有代码复用，这里出现了一个奇怪的try： https://github.com/yejue/nonebot-plugin-with-ai-agents/blob/1c1b96d76d115f90b06b40e1697b2c5b8517cdef/llm/llms/openai.py#L54-L65 把访问大模型放在代码放在 try 外面，可能不是你的本意
yejue
问题
经过一段时间的聊天，Bot 开始回复消息”有错误，自己看日志，可能过期了“。 观察日志发现：
插件版本
0.1.14
Nonebot版本
2.3.0
python 版本
3.12.3
复现过程
只要启动 Bot ，聊一段时间就会出现这样的问题。
日志上下文：
截图
文字
排查
已确定不是由网络问题引发的
https://github.com/yejue/nonebot-plugin-with-ai-agents/blob/1c1b96d76d115f90b06b40e1697b2c5b8517cdef/llm/llms/dashscope.py#L60-L70 通过观察源码发现这个错误是直接通过错误基类被捕获的，然后以 str 的形式与 r.text 拼接，在这里这个异常的str是'output'，然后 bot 回复 ”有错误，自己看日志，可能过期了“。这是不是有点太草率了。
把 body 的内容打印出来以后，发现 role 的内容也是正常的。