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

Improve series processing #3691

Open
paranoidi opened this issue Feb 16, 2023 · 8 comments
Open

Improve series processing #3691

paranoidi opened this issue Feb 16, 2023 · 8 comments
Assignees

Comments

@paranoidi
Copy link
Member

The series plugin should be able to process entries by learning them into series db. As discussed on chat.

Solution:

  • Deprecate parse_only option
  • Introduce new do option which takes list of operations done

Example:

do: [parse]
do: [parse, learn]
do: [parse, learn, accept, reject]  # default value

Values accept, reject might be combined into single option "actions" ?

In future, this can be expanded into verifying successful download in local filesystem when doing sorting task.

  • Add new verified column (datetime)
  • Add new do option verify that marks current datetime there
  • Add new series option verify: 10 hours after which new download is allowed.
@paranoidi
Copy link
Member Author

@Ram-Z ping

@github-actions
Copy link

This issue is stale because it has been open 150 days with no activity. Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the Stale label Jul 17, 2023
@github-actions
Copy link

This issue has been stale for 30 days and is being closed.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 16, 2023
@gazpachoking
Copy link
Member

do: [parse]
do: [parse, learn]
do: [parse, learn, accept, reject]  # default value

Are there instances where we wouldn't want the previous modes? i.e. would we ever want 'learn' without 'parse'? Or the filtering without learn? Maybe it could just be mode: parse|learn|filter

@gazpachoking gazpachoking reopened this Aug 16, 2023
@paranoidi
Copy link
Member Author

@gazpachoking yes, spesifically the future plan mentioned do: [parse, verify] in the sorting task

@gazpachoking
Copy link
Member

Oh, I missed the whole verify idea, that's interesting. I assume the use of that would be to somehow make a task that forgets episodes that haven't been verified after some time?

@github-actions github-actions bot removed the Stale label Aug 17, 2023
@paranoidi
Copy link
Member Author

Something like that or make series plugin option that ignores unverified downloads after some period of time. Also CLI could show verified status, not just download status.

Copy link

This issue is stale because it has been open 150 days with no activity. Remove stale label or comment or this will be closed in 30 days.

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

No branches or pull requests

3 participants