Skip to content

CMRT 33: Track table CMRT column polish, hotcue copy/add, and manual canonical re‑election - #48

Open
Swarnadip-Kar wants to merge 7 commits into
cmrt-use-cmrt-beatgridfrom
cmrt-hotcue-overlay-column-reelection
Open

CMRT 33: Track table CMRT column polish, hotcue copy/add, and manual canonical re‑election#48
Swarnadip-Kar wants to merge 7 commits into
cmrt-use-cmrt-beatgridfrom
cmrt-hotcue-overlay-column-reelection

Conversation

@Swarnadip-Kar

Copy link
Copy Markdown
Owner

Adds the user‑facing CMRT features to the library view and track menu:

  • A CMRT column that shows the grouping ID (padded to 4 digits), a colour‑spectrum swatch representing match percentage, and a “No CMRT group” label for solo (single‑member) groups.
  • Custom sorting of the CMRT column so that grouped tracks sort together in numeric order.
  • Copy/Add CMRT Hotcues actions in the Hotcues submenu, with the existing overlay‑active gate.
  • Disable Clear > Hotcues while a CMRT overlay is active, matching the safety gate already used by the hotcue copy actions.
  • “Make Canonical (CMRT)” action in the Analyze menu that lets the user manually promote a non‑canonical member to canonical, reusing the same re‑election logic the automatic AcoustID path uses.
  • Now live decks are refreshed directly - after re-election.

Depends on: cmrt-use-cmrt-beatgrid (#47)

@Swarnadip-Kar
Swarnadip-Kar force-pushed the cmrt-hotcue-overlay-column-reelection branch from 62b228a to 363fe6f Compare July 11, 2026 16:53
@Swarnadip-Kar
Swarnadip-Kar force-pushed the cmrt-hotcue-overlay-column-reelection branch from 363fe6f to 6f4b0e9 Compare July 11, 2026 17:37
@coveralls

coveralls commented Jul 11, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29161956439

Coverage decreased (-0.1%) to 30.068%

Details

  • Coverage decreased (-0.1%) from the base build.
  • Patch coverage: 424 uncovered changes across 10 files (5 of 429 lines covered, 1.17%).
  • 16 coverage regressions across 6 files.

Uncovered Changes

File Changed Covered %
src/widget/wtrackmenu.cpp 157 0 0.0%
src/musicbrainz/cmrtgroupingservice.cpp 52 0 0.0%
src/library/dlgtrackinfo.cpp 47 0 0.0%
src/track/track.cpp 44 0 0.0%
src/library/basesqltablemodel.cpp 38 1 2.63%
src/library/tabledelegates/cmrtdelegate.cpp 29 0 0.0%
src/library/dao/trackdao.cpp 24 1 4.17%
src/library/basetracktablemodel.cpp 22 0 0.0%
src/library/basetracktablemodel.h 12 0 0.0%
src/library/librarytablemodel.cpp 4 3 75.0%

Coverage Regressions

16 previously-covered lines in 6 files lost coverage.

File Lines Losing Coverage Coverage
src/analyzer/trackanalysisscheduler.cpp 4 64.74%
src/widget/wtrackmenu.cpp 4 0.0%
src/musicbrainz/cmrtgroupingservice.cpp 3 0.0%
src/library/basetracktablemodel.cpp 2 8.16%
src/library/dlgtrackinfo.cpp 2 0.0%
src/library/dao/trackdao.cpp 1 44.05%

Coverage Stats

Coverage Status
Relevant Lines: 117840
Covered Lines: 35432
Line Coverage: 30.07%
Coverage Strength: 58452.49 hits per line

💛 - Coveralls

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants