[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:29:08 EEST 2019



> 在 2019年5月17日,下午1:20,Gyan <ffmpeg at gyani.pro> 写道:
> 
> 
> 
> On 17-05-2019 10:36 AM, Liu Steven wrote:
>> 
>>> 在 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.
> 
> Consider your patch NAKed. I created a new patch with new prose. I used the URL you provided for which you were credited.
I don’t want waste my time on this too. NAK
> 
> I'm not going to waste any more time on this.
> 
> Gyan
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".





More information about the ffmpeg-devel mailing list