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

upd: [JP] asset COMMISSION_START #4353

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

guoh064
Copy link
Contributor

@guoh064 guoh064 commented Nov 4, 2024

群友提供:
报错日志1730722503176.zip
之前只裁剪文字部分的时候裁剪得过于激进,导致灰色背景被识别为COMMISSION_START,故有此commit。

@LmeSzinc
Copy link
Owner

LmeSzinc commented Nov 5, 2024

对 COMMISSION_START 增加颜色检查

@LmeSzinc LmeSzinc added the bug / 缺陷 Something isn't working label Nov 5, 2024
@Air111
Copy link
Contributor

Air111 commented Nov 5, 2024

match_appear_on吧?

@guoh064
Copy link
Contributor Author

guoh064 commented Nov 7, 2024

啊,颜色检查不是数颜色吗,这几个区别是什么

@LmeSzinc
Copy link
Owner

参考

def strategy_has_mob_move(self):
"""
Pages:
in: STRATEGY_OPENED
out: STRATEGY_OPENED
"""
if (self.appear(MOB_MOVE_ENTER, offset=MOB_MOVE_OFFSET)
and MOB_MOVE_ENTER.match_appear_on(self.device.image)):
return True
else:
return False

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug / 缺陷 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants