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

关于分文件保存弹幕文件的优化方案征集 #148

Closed
MiKoto-Railgun opened this issue Feb 2, 2025 · 3 comments · Fixed by #153
Closed

关于分文件保存弹幕文件的优化方案征集 #148

MiKoto-Railgun opened this issue Feb 2, 2025 · 3 comments · Fixed by #153

Comments

@MiKoto-Railgun
Copy link
Contributor

现在的保存弹幕文件的命名是简单的danmaku1.json,是否考虑加入episodeId,如danmaku_episodeId_1.json之类的,这样的话在本来开了一个mpv窗口情况下再打开一个新的窗口,就不会有弹幕文件错乱或加载出错了

另外,现在的弹幕加载逻辑中,若是之前手动添加了弹幕,那么重新打开后会先自动加载上次匹配的弹幕,但会在匹配完弹幕库后就显示加载了多少弹幕,而之后手动添加弹幕加载还会重新显示加载了多少弹幕,并调用load_danmaku使得出现显示两次弹幕数量和加载卡顿的情况

@Tony15246
Copy link
Owner

现在的保存弹幕文件的命名是简单的danmaku1.json,是否考虑加入episodeId,如danmaku_episodeId_1.json之类的,这样的话在本来开了一个mpv窗口情况下再打开一个新的窗口,就不会有弹幕文件错乱或加载出错了

个人感觉开了一个mpv窗口情况下再打开一个新的mpv窗口已经不能算是插件的正常使用场景了。如果我没记错的话,同时打开的两个mpv窗口,插件之间的变量值都是相同的。想要强行区分两个mpv窗口,兼容多窗口播放场景,会为插件平添不必要的复杂检查机制,引入更多bug的隐患。我的个人想法是不考虑此种使用场景。

@Tony15246
Copy link
Owner

另外,现在的弹幕加载逻辑中,若是之前手动添加了弹幕,那么重新打开后会先自动加载上次匹配的弹幕,但会在匹配完弹幕库后就显示加载了多少弹幕,而之后手动添加弹幕加载还会重新显示加载了多少弹幕,并调用load_danmaku使得出现显示两次弹幕数量和加载卡顿的情况

这个问题确实有改善的必要,并且进行改进的理论可行性很高。等解决完插件目前的问题,可以考虑优化这一点

@dyphire
Copy link
Collaborator

dyphire commented Feb 2, 2025

添加 episodeId 是不必要的,但针对 mpv 多窗口的使用场景可以通过添加进程 pid 来区分

@Tony15246 Tony15246 linked a pull request Feb 3, 2025 that will close this issue
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 a pull request may close this issue.

3 participants