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

异步化AutoBangumi,实现部分功能 #825

Open
wants to merge 36 commits into
base: aio-refactor
Choose a base branch
from

Conversation

shininome
Copy link
Contributor

具体改动如下

downloader 部分

  1. 没有提供一个 add_torrents 而是全部由addtorrent实现, 因为暂时没有找到怎么实现一次上传多个文件

没有实现TR.

renamer部分

  1. 全新的 rawparser, 效果也很一般,就是之前没事就报错我又想重写一下看看,想法是最大努力返回集数,因为名字和季度都可以手动修改,与其报错不下载,不如下载了慢慢纠错
  2. TMDB,MIKAN 现在统一返回Bangumi,

notification

  1. 加入了一个message字段,这样当没有episode的时候可以简单的发送其他消息,但是没有实现发送其他消息
  2. 仅验证了telegram, 其余渠道没有验证是否可行

api部分

实现了部分,但是delete bangumi后处理torrent没有实现,

  1. 搜索模块现在是全部搜索后返回, 速度比较了一下和之前返回的就差了1秒,就没有继续改动

renamer

1.现在是从downloader和database 获取,必须要downloader 有并且databased的downloaded字段为0,才会重新命名,对qb获取的种子数量加了限制,50,暂时没有提供更改方法

已知bug renamer失败了downloader不会返回什么数据,无法判断是否真正的重命名了
删除坏种也没有实现,不知道怎么弄,因为没有判断

  1. 实现偏移

但是在手动修改之后会出先偏移累加

rss

  1. bangumi 判断对于非聚合单独提了出来,搜索的现在不会因为名字改动就不搜索了(ANI!)

现在依旧要两次获取bangumi, 后面要改动一下

core

  1. 现在只是简单的循环,没有考虑什么别的, 需要后期的修改

engine

  1. 现在取消了engine 对于database的继承, database的引用在内部用 with 包裹保证安全.
    engine与database的行为逻辑并不相同, engine更多的是使用database, 我认为这样继承并不合适

其实现在大部分的继承都用组合了, 感觉抽象能力不够,没有办法给engine抽象出来一个基类

其他

  1. 屏蔽了httx的info消息,太吵了.
  2. 一些commit没有实现,只是简单的适配了一下现在的网页
  3. config太大会卡死

Copy link

vercel bot commented Aug 3, 2024

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 Sep 1, 2024 3:17pm

fix
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.

1 participant