DDMAL / DDMAL/cantus

Merge Liber Usualis local-deployment bug fixes (#981) before production indexing

Open
#984 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
16
Forks
4
Avg merge
2d 16h
Merged PRs (30d)
3

Description

#981 fixes a real bug (#979) where MEIParser silently drops every staff but the first on multi-staff pages -- 67% of the Liber Usualis corpus's 2318 pages have 2+ staves (up to 16 on one page). Without this fix, production indexing of the Liber Usualis would silently produce incomplete OMR data, the same way it did locally before the fix.

This needs to land (and any review feedback addressed) before real production indexing work in this migration (see the "Run production OMR indexing" sub-issue) can produce correct results.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.