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

some new features, improvements and config options #67

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

Tackyou
Copy link

@Tackyou Tackyou commented Jan 8, 2023

  1. dependency updates

  2. new config options
    DEBUG_RUN to quickly write decisions.log
    SCORE_VIDEOBITRATE
    SCORE_AUDIOCHANNELS
    REQUIRED_TO_ALLOW_PROCESSING one of the files has to contain one of the given parameters to allow processing, for example, to only remove files if a dual audio version is available (require .DUAL. or .MULTI. in one of the files) or only if a good quality version is available (require 1080p)

  3. better regex for deletefiles.sh, previous one wouldn't match all filenames properly

  4. a run script that can be used for full automation (cronjob)

  5. made the console output easier to read by shortening output and using emojis
    ✨ successful deletion
    ⚠️ failed deletion
    ✅ keep file
    ☑️ ignore/skip file
    ❌ delete file
    🆔 file id

Screenshot 2023-01-08 at 14 32 04

  • Something that might be an issue
    shortenedFilePath = '/' + '/'.join(part_info['file'][0].split('/')[-3:]) will cut out the last 3 parts of the path.
    For example /path/to/the/mount/Movies/stuff.mkv would show as /mount/Movies/stuff.mkv
    This is supposed to make the output easier to read by removing uninteresting stuff from the path, but if the path is less than 3 folders deep it will most likely throw an error.
    It probably also doesn't work on Windows Plex servers due to the backward slashes.
    Maybe remove this part before merging. (easiest way is to change it to shortenedFilePath = part_info['file'][0])

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.

3 participants