perf: 优化仓库识别速度 #8113
Merged
perf: 优化仓库识别速度 #8113
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
ambiguous
| if (need_search) { | ||
| if (!serch_item(roi,get_match_begin_pos()-2)) | ||
| continue; | ||
| } |
There was a problem hiding this comment.
是不是应该单独开一个循环遍历 m_all_items_roi 然后记下序号, 防止找了一遍完全没找到的情况 (
There was a problem hiding this comment.
如果完全没找到是直接退出报错，还是再找一遍。现在找的条件是得分大于0.85，还不是找的最后一个，最后一个术士和近卫芯片会匹配上。后面的芯片匹配得分貌似都大于0.8（。这个地方有什么好的想法吗。。。
There was a problem hiding this comment.
琢磨了一下，貌似能把上一页最后一个roi的内容临时存在一个Mat中，通过这个Mat来找。不知道能不能行，下午开服试试
目前的思路是按顺序匹配88种材料，定位上一页最后一个来确定当前页从哪个材料开始匹配。好处是匹配的次数少，坏处是定位不准炸了（ |
github-actions
bot
removed
the
ambiguous
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
识别时间缩短到10多秒。芯片材料中的近卫和术士芯片太过相近可能会导致识别不全和不匹配。