[FFmpeg-devel] [PATCH v2] doc/filters: add the sr filter model generation scripts new repository link

Liu Steven lq at chinaffmpeg.org
Fri May 17 08:06:44 EEST 2019



> 在 2019年5月17日,下午12:46,Gyan <ffmpeg at gyani.pro> 写道:
> 
> 
> 
> On 17-05-2019 03:14 AM, Steven Liu wrote:
>> 
>>> 在 2019年5月17日,02:57,Gyan <ffmpeg at gyani.pro> 写道:
>>> 
>>> 
>>> 
>>> On 16-05-2019 11:13 AM, Steven Liu wrote:
>>>> Hold on the old repository link and mention new repository link  development continues
>>>> 
>>>> Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
>>>> ---
>>>>  doc/filters.texi | 3 ++-
>>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>> 
>>>> diff --git a/doc/filters.texi b/doc/filters.texi
>>>> index 38c70bf674..2e9db150f2 100644
>>>> --- a/doc/filters.texi
>>>> +++ b/doc/filters.texi
>>>> @@ -16538,7 +16538,8 @@ See @url{https://arxiv.org/abs/1609.05158}.
>>>>  @end itemize
>>>>    Training scripts as well as scripts for model generation are provided in
>>>> -the repository at @url{https://github.com/HighVoltageRocknRoll/sr.git}.
>>>> +the repository at @url{https://github.com/HighVoltageRocknRoll/sr.git}(have stop maintaince)
>>>> +or @url{https://github.com/XueweiMeng/sr/tree/sr_dnn_native}(development continues).
>>>>    The filter accepts the following options:
>>>>  
>>> Made some changes and pushed as 6044534964a463663803b590fe66a212af4b3834
>> I think you can review and comment the patch, then i can modify it.
>> but i don’t think you just leave a message is good:
>> 
>>>> commit c2771bbf815bd7d0ff273ba590b7db02a303399f
>> Author: Gyan Doshi <ffmpeg at gyani.pro>
>> Date:   Fri May 17 00:22:04 2019 +0530
>> 
>>     doc/filters: update URL for sr filter script repository
>> 
>>     Thanks to Steven Liu for the update.
>> 
>>> 
> It's a simple URL update for which I had to rewrite the prose, so maintaining authorship didn't make sense.
No, I’m the patch Author, not simple or complex, you have change the author infomation to you, that is wrong. you can comment or tick me to change it.
And the new URL is Xuewei’s github, I can take responsibility for this change, the change will continue the DNN native, it means the DNN native will supprt not only SR, also support derain.

> You should have posted a message - not reverted and especially not pushed.
I do that same as you, and Simple Change. If you don’t change the Author infomation, i would not revert that.

Even you are the mainatiner of the Documentations, but I don’t think you should change any author info of the patches.

Thanks

Steven






More information about the ffmpeg-devel mailing list