Feature: Auto measure label handling

[riedde]

Better handling of Measure labels

• joining labels when multiple measures pointing to same zone (avoids labels like 1/9)
• also works with already joined labels (multiRests)
• introducing diacritical labels (supplied and del)

needs:

• [x] code clean up
• [x] introducing functions (avoid redundancy)

[bwbohl]

@riedde what’s the status of this draft, any chance to resolve conflicts and close TODOs in the near future?

[riedde]

@bwbohl conflicts resolved. I'll have a look at the TODOs

[bwbohl]

What’s an appropriate dataset for testing this?

[riedde]

@bwbohl e.g., https://klarinettenquintett.weber-gesamtausgabe.de/index.html?lang=de (Erstdruck 1)

[bwbohl]

Tested with the klarinettenquintett dataset:

• label seem correct but Concordance navigation fails ,e.g. On Baermann D+-st with the multirest in Allegro/Klarinette M.1–11 ff.

[bwbohl]

The multiRest at the start of the Allegro in D+-st is also missing "–11"

[krHERO]

ping @nikobeer

[bwbohl]

pinging @riedde

[riedde]

@bwbohl I reviewed the function measure:getMeasureLabel(). It seems that there was an else case missing (added at https://github.com/Edirom/Edirom-Online/pull/302/commits/14837c2761d9d49c583c69571af04e029d7e6a14). I'm not sure if this coveres all cases but it is much more better than before and works with several datasets. I can offer you my dataset for testing if you like (three editions).

[riedde]

While testing I noticed that there is a bug when loading the measure based view with parts. I have to investigate this.

[bwbohl]

does this also address #432 ?

[riedde]

@bwbohl I don't think that this addresses #432, directly.