hiroi-sora / Umi-OCR_plugins

Umi-OCR 插件库
MIT License
318 stars 32 forks source link

[OCR引擎增加]添加微信ocr本地离线识别 #16

Closed eaeful closed 1 week ago

eaeful commented 1 week ago

我自己电脑和windows 10虚拟机测试是正常的,不知道还有没有其他bug没有发现,

hiroi-sora commented 1 week ago

感谢贡献,我会采纳此PR。不过我提一些建议,希望改一下:

  1. 可以参考 PR-3 ,将你的仓库作为 Submodule 提交给本仓库,链接到本仓库的 WechatOCR_umi_plugin 目录。这样维护更方便。
  2. 如果第三方库 third_party_libs 的体积不大(都是些纯文本文件)的话,那么建议一并提交到你的仓库里。如果体积大,则可以你的readme中说明一下开发构建流程。
  3. 建议不要区分内置exe手动填写exe路径两个版本的代码。是否可以用一份代码实现这两种模式?比如用户不填写 wechat_ocr_dir 的话,则从插件目录中搜索内置的exe;如果用户已填写,则优先用用户指定的exe。发布Release时,可以发布两个版本的包;两个包都用同一份代码,区别只是 是否内置exe。
  4. start() 时,启动失败会返回 return f"[Error] 启动失败:{str(e)}" 。如果失败原因是exe不存在或版本不兼容,请确保 str(e) 明确指出了失败原因,并提醒用户检查路径是否正确。
eaeful commented 1 week ago

代码修改了,弄成用一份代码实现这两种模式了,Release也变了,.gitmodules也把链接加进去了

hiroi-sora commented 1 week ago

OK,感谢贡献。

你的仓库最好再加个 .gitignore 文件,将不必要的临时文件(缓存)等排除掉,比如

**/__pycache__

然后手动将已经加入仓库的临时文件给删掉。

eaeful commented 1 week ago

ok,我已经把pycache相关的删了,加了.gitignore 文件,但是我看这个插件库内的.gitmodules,虽然有添加了[submodule "WechatOCR_umi_plugin"]这串代码,但是首页没有 WechatOCR插件给链接进插件仓库内,不知道怎么回事