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

feat: add kisssub dense torrent collect #651

Open
wants to merge 13 commits into
base: 3.2-dev
Choose a base branch
from

Conversation

TrueRou
Copy link

@TrueRou TrueRou commented Dec 25, 2023

前言

#647 支持解析包含多个视频的种子, 进而支持合集下载

方案简述

  • 引入Dense概念, 用来描述一个种子中包含多个视频的番剧合集
  • 引入DenseCollector, SearchDenseTorrent, DenseRSSAnalyser并实现
  • 添加kisssub搜索源, Dense搜索默认使用kisssub

后续

  • 可能需要对前端进行一些改造, 让用户更好明白自己使用的是合集搜索
  • 尚不清楚Renamer是否足够强大处理各种各样的合集种子

Copy link

vercel bot commented Dec 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
auto-bangumi ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 26, 2023 1:39pm

@EstrellaXD
Copy link
Owner

请按照 https://github.com/EstrellaXD/Auto_Bangumi/blob/main/CONTRIBUTING.md 开发,不过这个 RFC 不是已经被关闭了吗

@EstrellaXD
Copy link
Owner

稍微看了一眼 PR 代码。下次做比较大的变更之前还是先用 RFC 讨论再动手比较好。

  • 目前开发工作在 3.2-dev 以及各个功能分支,其中请求库已经变更为 httpx 的协程方式。
  • 希望不要变动顶层 API,具体判断交给类中的功能会比较好
  • 引入的合集解析可以直接集成在 TitleParser 中,而非单独拆分出一类,这会让维护变得困难。其他搜索收集功能也是一样的。

感谢 PR,不过还是希望可以按照当下的 Roadmap 和开发进度进行修改,希望下次可以讨论交流之后再着手开发。

@EstrellaXD EstrellaXD changed the base branch from main to 3.2-dev December 25, 2023 09:33
@TrueRou
Copy link
Author

TrueRou commented Dec 25, 2023

稍微看了一眼 PR 代码。下次做比较大的变更之前还是先用 RFC 讨论再动手比较好。

  • 目前开发工作在 3.2-dev 以及各个功能分支,其中请求库已经变更为 httpx 的协程方式。
  • 希望不要变动顶层 API,具体判断交给类中的功能会比较好
  • 引入的合集解析可以直接集成在 TitleParser 中,而非单独拆分出一类,这会让维护变得困难。其他搜索收集功能也是一样的。

感谢 PR,不过还是希望可以按照当下的 Roadmap 和开发进度进行修改,希望下次可以讨论交流之后再着手开发。

感谢review, 回复中提到的一些问题已经调整. 之前那个RFC已经reopen并且做了一些修改.

确实跟Roadmap和开发进度有些不符, 可以深入交流一下再继续修改或者未来再议

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