Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

【bug】部分问题和改进建议 #94

Open
Agnes4m opened this issue Nov 5, 2023 · 3 comments · Fixed by #95
Open

【bug】部分问题和改进建议 #94

Agnes4m opened this issue Nov 5, 2023 · 3 comments · Fixed by #95
Labels
enhancement New feature or request

Comments

@Agnes4m
Copy link

Agnes4m commented Nov 5, 2023

1、设置的默认github代理失效,从而启动时一定因为这个错误而卡主20秒左右
2、pillow10.0.0版本会抛弃getsize用法,从而导致插件失效
3、建议使用nonebot_plugin_saa来实现多适配器的抽签功能,而舍弃只使用onebot.v11的用法,使得适配全平台(管理权限暂时没用统一写法,默认全平台的功能暂时只有抽签

@KafCoppelia
Copy link
Member

  1. 采纳,通过添加用户配置项 GITHUB_PROXY 解决很优雅
  2. 关于 pillow,插件依赖里指定的是 ^9.0.0,升不到10,但代码可以做出修改
  3. 比较关心的是是否使用 nonebot_plugin_saa,影响原有的功能是一方面,其次插件内有很多有关输入内容检查的。目前看 nonebot_plugin_saa 的文档真是蜻蜓点水,综合来看要使用它的成本有点高。

题外话,我手头没有可用于测试的bot,目前测试维护的插件用的 nonebug ,有一定局限,老哥有空可以帮我测试一下其他的插件 tarot v0.5.0a3

@KafCoppelia KafCoppelia added the enhancement New feature or request label Nov 5, 2023
@Agnes4m
Copy link
Author

Agnes4m commented Nov 5, 2023

nonebot_plugin_saa我使用这个功能大概有4个月了,几乎没出现过任何问题。唯一一次问题是在于nonebot2删除了qqguild适配器,在我提出iss后1小时内就修复了这个bug,可以说更新非常及时。

文档似乎是上个月才创的,主要是功能确实非常的简单没啥好描述的,仅仅看readme就可以完全掌握用法,这也是该插件的核心思想,用最简单的方法去构建不同适配器的消息发送,也就是说只要使用saa,未来如果saa适配了新的适配器(例如discord等),那么只需要更新saa插件,自己本体nb插件是不需要更新,便可以完成新适配器的增加,我个人认为是非常方便的。


至于塔罗牌这个。。正常使用没啥问题,还是github代理的问题,因为ghproxy.com基本算是寄了,个人认为同方法让用户自行填写代理为好。

如果按我建议采取多适配器方法,则无需采用转发功能(除了v11其他都不限制机器人),只需要添加消息所在适配器判定即可

@SaltyFishFly
Copy link

utils.py中 71 行改为:

# func drawing
bbox = ttfront.getbbox(title)
font_length = (bbox[2] - bbox[0], bbox[3] - bbox[1])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants