[FEATURE] feat(ripper): sort main feature by chapters, filesize, then…#1698
Merged
microtechno9000 merged 4 commits intoautomatic-ripping-machine:mainfrom Mar 12, 2026
Merged
[FEATURE] feat(ripper): sort main feature by chapters, filesize, then…#1698microtechno9000 merged 4 commits intoautomatic-ripping-machine:mainfrom
microtechno9000 merged 4 commits intoautomatic-ripping-machine:mainfrom
Conversation
… track number (automatic-ripping-machine#1697) * feat(ripper): sort main feature by chapters, filesize, then track number Updated main feature scan to now look at chapters and filesizes. Updated database to keep track of these new values as well. Sorts tracks by highest chapter count, then largest filesize, then lowest track number to better identify the main feature on Disney Blu-rays. Fixes automatic-ripping-machine#1697 * Update VERSION
uprightbass360
added a commit
to uprightbass360/automatic-ripping-machine-neu
that referenced
this pull request
Mar 3, 2026
…ping-machine#1605, automatic-ripping-machine#1660 PR automatic-ripping-machine#1698 — Main feature chapters/filesize sorting: - Add chapters (Integer) and filesize (BigInteger) to Track model - Parse MakeMKV TINFO fields 8 (chapters) and 11 (filesize) - Sort main feature by chapters desc, length desc, filesize desc, track_number asc — fixes Disney Blu-ray playlist obfuscation - Add BigInteger to database layer exports - Migration: a4b5c6d7e8f9 PR automatic-ripping-machine#1605 — TV series disc label parsing: - Add parse_disc_label_for_identifiers() for labels like STARGATE_ATLANTIS_S1_D2 → S1D2 - Add normalize_series_name() and get_tv_folder_name() - Integrate into Job.build_final_path() for TV series - Config keys: USE_DISC_LABEL_FOR_TV, GROUP_TV_DISCS_UNDER_SERIES - Migration: b5c6d7e8f9a0 PR automatic-ripping-machine#1660 — Config file ownership fix: - Replace cp --no-clobber with install -o arm -g arm -m 644 in arm_user_files_setup.sh to set ownership atomically Includes 33 new tests covering all three features.
3 tasks
uprightbass360
added a commit
to uprightbass360/automatic-ripping-machine-neu
that referenced
this pull request
Mar 4, 2026
…ping-machine#1605, automatic-ripping-machine#1660 PR automatic-ripping-machine#1698 — Main feature chapters/filesize sorting: - Add chapters (Integer) and filesize (BigInteger) to Track model - Parse MakeMKV TINFO fields 8 (chapters) and 11 (filesize) - Sort main feature by chapters desc, length desc, filesize desc, track_number asc — fixes Disney Blu-ray playlist obfuscation - Add BigInteger to database layer exports - Migration: a4b5c6d7e8f9 PR automatic-ripping-machine#1605 — TV series disc label parsing: - Add parse_disc_label_for_identifiers() for labels like STARGATE_ATLANTIS_S1_D2 → S1D2 - Add normalize_series_name() and get_tv_folder_name() - Integrate into Job.build_final_path() for TV series - Config keys: USE_DISC_LABEL_FOR_TV, GROUP_TV_DISCS_UNDER_SERIES - Migration: b5c6d7e8f9a0 PR automatic-ripping-machine#1660 — Config file ownership fix: - Replace cp --no-clobber with install -o arm -g arm -m 644 in arm_user_files_setup.sh to set ownership atomically Includes 33 new tests covering all three features.
microtechno9000
approved these changes
Mar 12, 2026
Collaborator
microtechno9000
left a comment
There was a problem hiding this comment.
Nice feature, good logs. LGTM
13dfc7f
into
automatic-ripping-machine:main
8 checks passed
|
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
… track number (#1697)
Updated main feature scan to now look at chapters and filesizes. Updated database to keep track of these new values as well.
Sorts tracks by highest chapter count, then largest filesize, then lowest track number to better identify the main feature on Disney Blu-rays.
New Feature - Updated main feature scan to now look at chapters and filesizes. Updated database to keep track of these new values as well.
Patch or Bugfix - [BUGFIX]
Description
Sorts tracks by highest chapter count, then largest filesize, then lowest track number to better identify the main feature on Disney Blu-rays.
Fixes #1697
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration
I have loaded American Sniper, The Incredibles, Tarzan, & Finding Nemo
Checklist:
Changelog:
Include the details of changes made here
arm/migrations/versions/edf2272c0a9d_track_chapters_filesize.pyto add track chapter sizes and track file sizes to DBarm/models/track.pyto load the values from the DBarm/ripper/makemkv.pyto read in the new values and sort by chapters first, then filesizes, then track numberarm/ripper/makemkv.pyto use 'minlength' value from the arm.yaml config file instead of defaulting to 0 (for consistency when ripping)arm/ripper/utils.pyto write new values into the DBLogs
Attach logs from successful test runs here
Tarzan:
tarzan.log
American Sniper:
American Sniper.log