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

[Feature Request] 非常好工具,就是还有点坏 #3

Open
ty114514 opened this issue Feb 12, 2024 · 1 comment
Open

[Feature Request] 非常好工具,就是还有点坏 #3

ty114514 opened this issue Feb 12, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@ty114514
Copy link

体验很棒,但是作为强迫症感觉还差那么一点,具体体现在:

  1. 内置字体也太太太大了(14.3 MB,占据了一半)
  2. 生成的 png 也太太太大了
  3. 速度欠佳(exec 调用慢为主,然后最后 draw 保存会卡一下)

和 Python 下的 https://github.com/amietn/vcsi 做了个简单对比,控制分辨率和矩阵一样大,时间戳均为开

# 4x5
# vcsi 1.38MB
real	0m4.416s
user	0m9.997s
sys	0m2.546s

# go-video-thumb 2.95MB
real	0m8.647s
user	0m15.840s
sys	0m1.491s
# 10x10
# vcsi 6.70MB
real	0m19.566s
user	0m45.752s
sys	0m7.692s

# go-video-thumb 14.42MB
real	0m46.895s
user	1m21.734s
sys	0m8.792s

感觉什么都不能解决的解决方案:

  1. 我觉得可以使用 https://github.com/adrg/sysfont 直接匹配用户设备上的字体
  2. 使用 jpg 有损压缩
  3. 思路暂无

最后,新年快乐!

@kmou424 kmou424 self-assigned this Feb 13, 2024
@kmou424 kmou424 added enhancement New feature or request good first issue Good for newcomers labels Feb 13, 2024
@kmou424
Copy link
Owner

kmou424 commented Feb 24, 2024

感谢你的关注。

关于你提出的三点问题,其实我也有考虑过:

  1. 一开始选用的是Go中自带的一个非常精简的小字体(精简到以字节数组形式放在font包里边的),只支持ASCII里的符号,但是考虑到文件名里可能会出现Unicode文字,最后选择了内置一个微软雅黑。你提出的也是一个不错的解决方案,毕竟用户也不会去对一个显示不出文件名的视频进行处理。
  2. 输出文件体积确实没有考虑到,因为我做这个小工具就是抱着偷懒的目的去做的,是想要保证清晰度的前提下预览整个视频。后续我会加上输出格式和压缩级别的命令行参数。
  3. 效率这块我还没有考虑过,可以在此贴下讨论一下。

回复有些晚,不好意思,祝元宵节快乐。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants