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

打印处理进度百分比信息 #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

tumuyan
Copy link

@tumuyan tumuyan commented Aug 18, 2022

因为目前没有打印进度百分比,在等待处理时难以预估进度和事件。
我对realcugan的处理事件进行分析后,发现主要耗时在stage2阶段。
因此针对几种stage2阶段的函数,增加了打印进度百分比的信息(可以近似看作整体处理进度)

@ArchieMeng
Copy link

打印进度Log会在下游视频处理软件中频繁Spamming。因此,之前就合过了移除sync gap阶段输出的PR #14
建议像nihui/realsr-ncnn-vulkan#41 一样,添加一个quiet参数,使得进度打印可以被控制,而不是一直开启。

@tumuyan
Copy link
Author

tumuyan commented Sep 18, 2022

打印进度Log会在下游视频处理软件中频繁Spamming。因此,之前就合过了移除sync gap阶段输出的PR #14 建议像nihui/realsr-ncnn-vulkan#41 一样,添加一个quiet参数,使得进度打印可以被控制,而不是一直开启。

我不太确认下游软件的处理方式,理论上正则匹配输出内容,当只输出进度信息时不处理,就完事了,而不是非怎样不行。
当然用开关处理也是一种方法。

@ArchieMeng
Copy link

我不太确认下游软件的处理方式,理论上正则匹配输出内容,当只输出进度信息时不处理,就完事了,而不是非怎样不行。 当然用开关处理也是一种方法。

单独处理stderr消息过滤进度信息会极大增加适配工作量,如果直接有个参数控制会方便许多,而且实现上不是很难,可以照着nihui/realsr-ncnn-vulkan#41 加个控制参数。
我说的软件就是Video2x.现在那边是直接用了加了quiet参数的版本去屏蔽RealSR进度输出的。

@tumuyan
Copy link
Author

tumuyan commented Sep 18, 2022

单独处理stderr消息过滤进度信息会极大增加适配工作量,如果直接有个参数控制会方便许多,而且实现上不是很难,可以照着nihui/realsr-ncnn-vulkan#41 加个控制参数。
我说的软件就是Video2x.现在那边是直接用了加了quiet参数的版本去屏蔽RealSR进度输出的。

添加参数确实并不麻烦,但是quiet参数的pr同样也没有合并,是不是一定要使用这个方式呢?
即使不过滤log,方法还是有很多,比如用log-level参数设置不同的log等级取代原有v参数,比如添加编译参数。

@ArchieMeng
Copy link

有道理。不过,选用初始化参数来控制一是可以在运行时控制,二是程序的输出本身就很简单,不需要用logging 来做,反而会复杂化。总而言之,控制方式还有待讨论吧。我了解了,这个PR 就先这样吧。

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.

2 participants