Skip to content

Feature/audio outerproducthmm#47

Merged
CarlosCancino-Chacon merged 10 commits intodevelopfrom
feature/audio-outerproducthmm
Feb 20, 2026
Merged

Feature/audio outerproducthmm#47
CarlosCancino-Chacon merged 10 commits intodevelopfrom
feature/audio-outerproducthmm

Conversation

@laurenceyoon
Copy link
Collaborator

@laurenceyoon laurenceyoon commented Feb 16, 2026

(Apologies for the big PR)

What is changed

  • New AudioOuterProductHMM: 2-layer HMM with sound/pause sub-states per score position. Uses 88-bin CQT harmonic mask template matching and spectral-flux-driven transitions. Uses CQTSpectralFluxProcessor feature (89-dim).
  • Unified score loading & evaluation refactor: Score loading now uses partitura.load_score() + merge_parts() (supports MusicXML, MEI, etc).
  • Simplified debug output: Debug now saves warping path TSV, results JSON, and a single alignment plot.
  • Asset updates: Renamed example files to simple_mozart_k265_var1.*, added MEI/match/annotation files for Mozart K265 and Bach BWV 858.

@laurenceyoon laurenceyoon force-pushed the feature/audio-outerproducthmm branch 2 times, most recently from 2afe571 to 55570f7 Compare February 17, 2026 06:56
@laurenceyoon laurenceyoon marked this pull request as ready for review February 17, 2026 07:14
@laurenceyoon laurenceyoon force-pushed the feature/audio-outerproducthmm branch from 55570f7 to 534b057 Compare February 17, 2026 08:05
Copy link
Collaborator

@CarlosCancino-Chacon CarlosCancino-Chacon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a couple minimal changes, but I otherwise we can merge this branch. Some things might need to be unified later, after all methods are there

@CarlosCancino-Chacon CarlosCancino-Chacon merged commit 33dad1b into develop Feb 20, 2026
8 checks passed
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