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.

Sign up for GitHub

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

Jump to bottom

正常词语被去叠字过滤 #65

Closed
rxia24 opened this issue Jan 25, 2020 · 4 comments
Closed

正常词语被去叠字过滤 #65

rxia24 opened this issue Jan 25, 2020 · 4 comments

Comments

@rxia24
Copy link
Contributor

rxia24 commented Jan 25, 2020

新年快乐，身体健康！

已确定config\text.json 中 modifier.deduplicate为 false，正常词语也被识别成叠字了。

如图所示：
image
@tinyAdapter
Copy link
Member

不应该啊……试过别的游戏了么？我这边测试是没问题的

有一个想法：可能是提取到的原始文本就被去掉了叠字……看看别的钩子？

@rxia24
Copy link
Contributor Author

rxia24 commented Jan 26, 2020 •
edited
Loading

搞清楚了，是这个游戏社团的游戏都会这样，
和你猜想的一致，有的钩子提取到的原始文本就去掉了叠字，有的原始文本就把每个文字重复一遍。

image

不过我把 modifier.deduplicate 改为true, 会把正常的字识别为叠字，或许更好的有办法？
image

@rxia24 rxia24 closed this as completed Jan 26, 2020
@CPCer
Copy link

CPCer commented May 7, 2020

@rxia24 可能还是游戏的问题？或者作者已经更新？我这边开了去叠字，不会把正常的重复去了……
不过我偶尔遇到整句重复，没去掉

@CPCer
Copy link

CPCer commented May 7, 2020 •
edited
Loading

噫，发现我的去叠字也会去正常的字，，打脸太突然……
您的issue也可以再开

Yesterday17 added a commit to Yesterday17/YUKI-Translator that referenced this issue Aug 10, 2020
@Yesterday17
增加指定去重字符数量
211876f 
project-yuki#65
Yesterday17 added a commit to Yesterday17/YUKI-Translator that referenced this issue Aug 10, 2020
@Yesterday17
增加指定去重字符数量
a57b0c3 
project-yuki#65
@Yesterday17 Yesterday17 mentioned this issue Aug 10, 2020
Open
Yesterday17 added a commit to Yesterday17/YUKI-Translator that referenced this issue Aug 10, 2020
@Yesterday17
feat: add deduplicateCount
e6f0bf6 
增加指定去重字符数量, Fix project-yuki#65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@tinyAdapter @CPCer @rxia24