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

增加了一个SRT合并切分插件 #826

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

X-T-E-R
Copy link

@X-T-E-R X-T-E-R commented Mar 20, 2024

增加了一个SRT合并切分插件,
并且修改了i18n让其能读取指定的文件夹

@RVC-Boss
Copy link
Owner

我康康

@X-T-E-R
Copy link
Author

X-T-E-R commented Mar 20, 2024

额外的一条对于喜欢分享自己整合包的up主/博主:请不要在合并到主支前把它做进整合包后抹去作者信息;虽然我不拦你

@v3ucn
Copy link
Contributor

v3ucn commented Mar 21, 2024

我那条视频没有声明代码来自官方PR? 你自己把别人的代码做进自己整合包,没有任何声明,然后阴阳别人,骂人,证据如下:

微信图片_20240321113506

微信图片_20240321114124

@X-T-E-R
Copy link
Author

X-T-E-R commented Mar 21, 2024

修正了上传文件的逻辑;增加了一个文件夹检测,现在不必要强制在前端gradio预览音频文件了

@X-T-E-R
Copy link
Author

X-T-E-R commented Mar 21, 2024

我那条视频没有声明代码来自官方PR? 你自己把别人的代码做进自己整合包,没有任何声明,然后阴阳别人,骂人,证据如下:

可能还是看法不同吧,我尊重您的看法,但是我认为“妈的”更多是一个情绪词而非骂人;并且之前那条github评论也只是说“请不要在合并到主支前把它做进整合包后抹去作者信息”

对于所谓的别人的代码做进自己的整合包,我在一开始就说明GSVI基于fast_inference_分支,但是并未提到CJ大佬是我的疏忽;

我已经重新在GSVI这个fork中声明,并且与ChasonJiang大佬进行了道歉

但是还请您不要超时空出警:fast_inference_分支3.11才被合并进了https://github.com/X-T-E-R/GPT-SoVITS-Inference ,但是我发的最近的一期分享推理包视频还是3月4日(详见https://space.bilibili.com/66633770)

It might just be a difference in perspective, and I respect your opinion. However, I view "妈的" more as an emotional expression rather than an insult. Moreover, my previous comment on GitHub merely stated, "Please do not integrate it into your bundle and remove the author's information before merging it into the main branch."

Regarding the incorporation of someone else's code into my own bundle, I did initially clarify that GSVI was based on the fast_inference_ branch, but I failed to mention CJ, which was an oversight on my part.

I have now made a declaration in the GSVI fork and have apologized to ChasonJiang.

But please, I urge you not to accuse me of something that's not true: the fast_inference_ branch was only merged into https://github.com/X-T-E-R/GPT-SoVITS-Inference on March 11, yet the latest video I posted about sharing the inference package was on March 4 (for more details, see https://space.bilibili.com/66633770).

@v3ucn
Copy link
Contributor

v3ucn commented Mar 21, 2024

我那条视频没有声明代码来自官方PR? 你自己把别人的代码做进自己整合包,没有任何声明,然后阴阳别人,骂人,证据如下:

可能还是看法不同吧,我尊重您的看法,但是我认为“妈的”更多是一个情绪词而非骂人;并且之前那条github评论也只是说“请不要在合并到主支前把它做进整合包后抹去作者信息”

对于所谓的别人的代码做进自己的整合包,我在一开始就说明GSVI基于fast_inference_分支,但是并未提到CJ大佬是我的疏忽;

我已经重新在GSVI这个fork中声明,并且与ChasonJiang大佬进行了道歉

但是还请您不要超时空出警:fast_inference_分支3.11才被合并进了https://github.com/X-T-E-R/GPT-SoVITS-Inference ,但是我发的最近的一期分享推理包视频还是3月4日(详见https://space.bilibili.com/66633770)

It might just be a difference in perspective, and I respect your opinion. However, I view "妈的" more as an emotional expression rather than an insult. Moreover, my previous comment on GitHub merely stated, "Please do not integrate it into your bundle and remove the author's information before merging it into the main branch."

Regarding the incorporation of someone else's code into my own bundle, I did initially clarify that GSVI was based on the fast_inference_ branch, but I failed to mention CJ, which was an oversight on my part.

I have now made a declaration in the GSVI fork and have apologized to ChasonJiang.

But please, I urge you not to accuse me of something that's not true: the fast_inference_ branch was only merged into https://github.com/X-T-E-R/GPT-SoVITS-Inference on March 11, yet the latest video I posted about sharing the inference package was on March 4 (for more details, see https://space.bilibili.com/66633770).

我每条视频简介都声明了来自官方pr,视频开头也声明了pr作者,只要有作者联系我声明版权,我会立刻删除相关视频和整合包内的代码,我的整合包里之前没有你的任何代码,也不可能出现你的任何代码,是你先阴阳别人,你自己拷贝别人代码进自己的整合包,没有任何声明,然后阴阳别人,你觉得你这么干挺友好?挺合乎逻辑?

@X-T-E-R
Copy link
Author

X-T-E-R commented Mar 21, 2024

我每条视频简介都声明了来自官方pr,视频开头也声明了pr作者,只要有作者联系我声明版权,我会立刻删除相关视频和整合包内的代码,我的整合包里之前没有你的任何代码,也不可能出现你的任何代码,是你先阴阳别人,你自己拷贝别人代码进自己的整合包,没有任何声明,然后阴阳别人,你觉得你这么干挺友好?挺合乎逻辑?

您好?我并不清楚自己是否一开始就”先阴阳别人“,但就我来说,一切的一切开始于下面这条github评论

额外的一条对于喜欢分享自己整合包的up主/博主:请不要在合并到主支前把它做进整合包后抹去作者信息;虽然我不拦你

您想要合并当然可以,不要抹去作者信息即可

对于您10点多突然加群,然后开始说我没有声明,我起来也是一激灵;因此后来认真听取您的意见,去和CJ大佬道歉并且重新著名在GSVI的readme中

我觉得您可能觉得冒犯的是,我在不知道与我私聊的就是您刘悦的情况下,在您的私聊框中说出了”我指的主要是刘悦“这句话,我为该句话对您造成的伤害进行公开道歉:对不起!

另外我保留了https://www.yuque.com/xter/zibxlp/swi1aw80nrlh270p 聊天记录,如果您有任何想补充和我说;该份聊天记录仅作事实记录,我对可能对您造成的冒犯深感抱歉。

@v3ucn
Copy link
Contributor

v3ucn commented Mar 21, 2024

我每条视频简介都声明了来自官方pr,视频开头也声明了pr作者,只要有作者联系我声明版权,我会立刻删除相关视频和整合包内的代码,我的整合包里之前没有你的任何代码,也不可能出现你的任何代码,是你先阴阳别人,你自己拷贝别人代码进自己的整合包,没有任何声明,然后阴阳别人,你觉得你这么干挺友好?挺合乎逻辑?

您好?我并不清楚自己是否一开始就”先阴阳别人“,但就我来说,一切的一切开始于下面这条github评论

额外的一条对于喜欢分享自己整合包的up主/博主:请不要在合并到主支前把它做进整合包后抹去作者信息;虽然我不拦你

您想要合并当然可以,不要抹去作者信息即可

对于您10点多突然加群,然后开始说我没有声明,我起来也是一激灵;因此后来认真听取您的意见,去和CJ大佬道歉并且重新著名在GSVI的readme中

我觉得您可能觉得冒犯的是,我在不知道与我私聊的就是您刘悦的情况下,在您的私聊框中说出了”我指的主要是刘悦“这句话,我为该句话对您造成的伤害进行公开道歉:对不起!

另外我保留了https://www.yuque.com/xter/zibxlp/swi1aw80nrlh270p 聊天记录,如果您有任何想补充和我说;该份聊天记录仅作事实记录,我对可能对您造成的冒犯深感抱歉。

你指控我没有任何问题,我做的不对的地方,直接说就行了,没有任何必要阴阳怪气,在技术交流分享平台上,你发的那句话很明显就是引战,明眼人都能看出来,也没必要解释。

@X-T-E-R
Copy link
Author

X-T-E-R commented Mar 21, 2024

image
image

增强了功能,增加了List合并小工具

@YYuX-1145
Copy link

很棒的设计方案。如果只是读srt的话,我不太建议调用第三方依赖,能少用就尽量少用,官方包的依赖有时候会跟不上版本的,而大多数用户又自己不会装依赖,这可能是个大麻烦

@wangyuanhao0918
Copy link

额外的一条对于喜欢分享自己整合包的up主/博主:请不要在合并到主支前把它做进整合包后抹去作者信息;虽然我不拦你

想锤别人,结果自己一屁股屎,哈哈,爱看

@LHRwudi
Copy link

LHRwudi commented Mar 22, 2024

@X-T-E-R 大家都提PR,给开源做贡献,怎么就你自我感觉良好?为什么别人一定要merge你的分支?你自己套个壳骗star就算了,还想让别人跟你一样?

@LHRwudi
Copy link

LHRwudi commented Mar 22, 2024

@X-T-E-R 刚看了一下,你自己的仓库全都拷贝的别人提的pr:TTS-for-GPT-soVITS 你不也抹去了作者的信息?pr作者你为什么不标注出来?你自己做贼心虚,还舔着个B脸说别人?

@X-T-E-R
Copy link
Author

X-T-E-R commented Mar 22, 2024

@X-T-E-R 刚看了一下,你自己的仓库全都拷贝的别人提的pr:TTS-for-GPT-soVITS 你不也抹去了作者的信息?pr作者你为什么不标注出来?你自己做贼心虚,还舔着个B脸说别人?

你看清楚再打好嘛,我现在有两个repo有关于gsv
一个https://github.com/X-T-E-R/GPT-SoVITS-Inference ,那是完全的fork
一个你发的那个链接 tts for gsv,那个明确的说明了是一个基于gsv的插件,是作为一个子模块存在的,他都不能独立运行

tts for gsv几乎全是我自己的代码(除了有一个classic Inference 使用了原版代码,但是那个仅作备用选项,默认不起用)

你真的能找到pr代码吗?(有一个流式传输,他有一个组装头是用到了pr代码,但我也在代码中标注了这一点)

gsvi相当规范的标注了主要是用了cj大佬的pr,你可以点开我的头像,找到这个repo

@LHRwudi
Copy link

LHRwudi commented Mar 22, 2024

classic Inference

你为什么不在readme里面声明?能在readme里面说清楚的事情,非得鬼鬼祟祟。classic Inference不起用你就可以不声明吗?谁规定的?你拷贝了原版代码就得声明,你懂什么叫开源协议吗?

@X-T-E-R
Copy link
Author

X-T-E-R commented Mar 22, 2024

classic Inference

你为什么不在readme里面声明?能在readme里面说清楚的事情,非得鬼鬼祟祟。classic Inference不起用你就可以不声明吗?谁规定的?你拷贝了原版代码就得声明,你懂什么叫开源协议吗?

谢谢您的建议,我马上去补充这一点

这是我的疏忽

@LHRwudi
Copy link

LHRwudi commented Mar 23, 2024

@X-T-E-R 看了一下所谓的聊天记录:https://www.yuque.com/xter/zibxlp/swi1aw80nrlh270p 好像是本来私聊沟通问题,结果你破防了,开始骂人,然后拉到大群,想让你的粉丝网暴别人,事实上你也达到目的,帮手挺多的,可惜啊,你不占理,最终只能赔礼道歉,笑死我了,这是自己锤自己?

@KOKOMI12345
Copy link

这里发生了什么吗?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants