[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