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

打印伐木统计数据 #398

Merged
merged 13 commits into from
May 18, 2024
Merged

打印伐木统计数据 #398

merged 13 commits into from
May 18, 2024

Conversation

atiasn
Copy link
Contributor

@atiasn atiasn commented May 14, 2024

  1. 统计木材的数量(包括多个木材),以及执行时间
  2. 可以自定义获取数量上限(最高2000)
  3. 首次时通过OCR识别统计数据,并保存
  4. 后续如果OCR识别到统计数据的关键字(“获取”),直接累加首次保存的数据

示例如下:
image

@huiyadanli huiyadanli requested a review from emako May 15, 2024 06:19
@emako
Copy link
Collaborator

emako commented May 15, 2024

image

@emako
Copy link
Collaborator

emako commented May 15, 2024

没有细节验证,是不是OCR位置不妥?对多种分辨率没适配。

这里有个rect_left_give_items_maybe的预测位置,不知道有没有帮助
https://github.com/GenshinMatrix/Fischless/blob/main/src/Desktop/Fischless.Fetch/Responsive/ResponsiveProvider.cs

@huiyadanli
Copy link
Member

huiyadanli commented May 15, 2024

我纯看代码,应该是按比例适配了位置区域的。

@atiasn
Copy link
Contributor Author

atiasn commented May 15, 2024

我看文档自动伐木是基于1920x1080 分辨率,我测试也是基于这个分辨率的。

估计你那儿OCR识别到内容,但是没有以下的字段。

// 会做如下的判断,不满足的算是为空,即未识别
return !string.IsNullOrEmpty(recognizedText) && 
           recognizedText.Contains("获得") &&
           (recognizedText.Contains('×') || recognizedText.Contains('x'));

我修改为Contains不用indexOf,可以在尝试下

rect_left_give_items_maybe 我研究下,或者扩大下OCR的位置区域。你测试是用什么分辨率。

没有细节验证,是不是OCR位置不妥?对多种分辨率没适配。

这里有个rect_left_give_items_maybe的预测位置,不知道有没有帮助 https://github.com/GenshinMatrix/Fischless/blob/main/src/Desktop/Fischless.Fetch/Responsive/ResponsiveProvider.cs

@huiyadanli
Copy link
Member

测试没问题,虽然降低了伐木效率,但是体验上了好几个档次。

看了下代码各种情况也考虑的挺全面的

感谢PR!

@huiyadanli huiyadanli merged commit cf16969 into babalae:main May 18, 2024
huiyadanli added a commit that referenced this pull request May 19, 2024
@atiasn atiasn deleted the wood-count branch May 24, 2024 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants