Auto_Bangumi icon indicating copy to clipboard operation
Auto_Bangumi copied to clipboard

feat: add kisssub dense torrent collect

Open TrueRou opened this issue 1 year ago • 4 comments

前言

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

方案简述

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

后续

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

TrueRou avatar Dec 25 '23 09:12 TrueRou

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

vercel[bot] avatar Dec 25 '23 09:12 vercel[bot]

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

EstrellaXD avatar Dec 25 '23 09:12 EstrellaXD

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

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

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

EstrellaXD avatar Dec 25 '23 09:12 EstrellaXD

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

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

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

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

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

TrueRou avatar Dec 25 '23 12:12 TrueRou