search

ArkMowers / arknights-mower

《明日方舟》长草助手
https://arkmowers.github.io/arknights-mower/
MIT License
505 stars 53 forks source link

refactor:重构公招代码 #645

Closed EightyDollars closed 1 month ago

ZhaoZuohong commented 1 month ago

方正雅宋是有版权的字体，不要提交到仓库里

ZhaoZuohong commented 1 month ago

recruit.pkl 是怎样生成的？以后再加新词条时怎样维护？

ZhaoZuohong commented 1 month ago

agent_token_first.png 或许截“首次获得”比较好？

ZhaoZuohong commented 1 month ago

recruit_config 不要通过参数传给 RecruitSolver，直接走 arknights_mower.utils.config

ZhaoZuohong commented 1 month ago

recruit_tags 方法中，把“根据词条计算招募结果”的逻辑与发邮件、识别等分开，为公招词条计算逻辑编写单元测试

EightyDollars commented 1 month ago

agent_token_first.png 或许截“首次获得”比较好？

因为高级凭证正好和xxx的信物会在同一个区域，而且只是为了防止在初次获得的时候识别不出是开包界面，所以只要能识别出是开包就行

EightyDollars commented 1 month ago

还差一个单元测试，明日再战

EightyDollars commented 1 month ago

recruit_tags 方法中，把“根据词条计算招募结果”的逻辑与发邮件、识别等分开，为公招词条计算逻辑编写单元测试

计算结果是一个单独的recruit_cal方法,然后我试着摸了一个单元测试这样