werrpy / vdator Goto Github PK

Right now, it only spellchecks commentary

Show appropriate errors when multiple video tracks or multiple chapter menus are present in mediainfo

will catch extra characters

https://github.com/werrpy/vdator/tree/mkvtoolnix

[CORRECT] Uses latest mkvtoolnix: 32.0.0 "Astral Progressions"

Research templates for Python

Use number emojis for errors (1-10)

https://github.com/werrpy/vdator/blob/master/vdator/checker.py#L399

bdInfo parsing should be separate from paste parsing. Make the paste parser only extract the bdinfo and mediainfo. Make a new bdinfo parser class/file.

Example:

[ERROR] Video track names missmatch:
BDInfo: MPEG-4 AVC Video / 24919 kbps / 1080p / 23.976 fps / 16:9 / High Profile 4.1
MediaInfo: MPEG-4 AVC Video / 24919 kbps / 1080p / 23,976 fps / 16:9 / High Profile 4.1

Use self.bdinfo and self.mediainfo instead

Use Google translation API
https://cloud.google.com/translate/docs/detecting-language#translate_detect_language-python

Construct filename from mediainfo and validate

Currently, mediainfo track title is compared against the bdinfo track.

The bot should check fields in the mediainfo and build a valid track name to compare against.

ambiguous
specify where it didn't find video track

Write some configuration files for settings instead of using env
Use env only for passwords, such as api keys

check settings to make sure they are adequate. bframes, ref, crf range, etc
Show compression stats

Process each separately

Check movie name and store if movie or tv show

https://pypi.org/project/guessit/

Can be used to detect presence of chapters
https://pastebin.com/FCmFNrUu

Right now, reactions are based on content of message. Use checker.get_report() instead.

subtitles
video and audio too?

construct_release_name() method uses a video_fps of 0, therefore it won't detect actually progressive tracks.

Should always be Chapter 01 instead of Chapter 1

if starts with chapter, expect chapter 01, etc.

https://pastebin.com/RtKHbXpR

Traceback (most recent call last):
  File "/home/user/vdator/venv/vdator/main.py", line 122, in on_message
    reply += checker.check_mkvmerge()
  File "/home/user/vdator/venv/vdator/checker.py", line 268, in check_mkvmerge
    mediainfo_version_name = re.search(version_name_regex_mediainfo, self.mediainfo['general'][0]['writing_application']).group(1)
AttributeError: 'NoneType' object has no attribute 'group'

Don't block message handler, parse in separate method

• Remove IMDb API as its glitchy
• Use TMDb's find() instead

_is_dvd() will crash if a bluray has

Video
Standard                                 : NTSC

Map video file title to codec

MPEG-H HEVC Video = HEVC
MPEG-4 AVC Video = AVC
MPEG-2 Video = MPEG-2

Get rid of the mediainfo offset.
Determine where audio codec starts and go from there

1080i 25fps is actually progressive, so should be p not i