[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **labels**: --> Fixed_2_21_0 - **status**: Started --> Fixed - **Patch**: push --> - **Comment**: commit 600c6f57051966a21ff55a3feabad1d57efd9ee0 Author: Malte Meyn Date: Sat Jan 13 01:30:39 2018 +0100 Issue 3208: compressed MMRs choose correct glyphs

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Patch**: countdown --> push - **Comment**: Patch counted down - please push. --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Tue Jan 16, 2018 01:20 PM UTC **Owner:** Malte Meyn

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- Description has changed: Diff: --- old +++ new @@ -1,4 +1,19 @@ -*Originally created by:* *anonymous +Issue 3208: compressed MMRs choose correct glyphs + +usable-duration-logs now affects only the choice of glyph for the +representation of a single measure. For multiple measures, glyphs

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Needs**: --> - **Patch**: new --> review - **Type**: --> Enhancement --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Sun Jan 14, 2018 12:18 PM UTC **Owner:** Malte Meyn **Attachments:**

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

Passes make, make check and a full make doc. Reg test diffs attached Attachments: - [2018-01-14-121546_1920x1080_scrot.png](https://sourceforge.net/p/testlilyissues/issues/_discuss/thread/99906edf/d9f8/attachment/2018-01-14-121546_1920x1080_scrot.png) (446.7 kB; image/png) -

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

Issue 3208: compressed MMRs choose correct glyphs usable-duration-logs now affects only the choice of glyph for the representation of a single measure. For multiple measures, glyphs for longer rests up to maxima are used. This is a second attempt to make it work and has little to do with my

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

I changed my mind: Allowing shorter rests doesn’t harm—no one *has* to use positive numbers in usable-duration-logs but if someone *really* wants to do it even though Gould et al. disadvise, we shouldn’t take away this functionality. This also means that we don’t need such a boolean grob

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

Would you really notate a rest in 3/8 as quarter or half instead of whole or dotted quarter? Apart from that, publisher 4 has anwered: Basically their answer confirms that we should support both versions. They do the same as publisher 2 for Bach/Handel and later; additionally they set a “1”

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

for default expand-limit I vote for 8. about usable-duration-logs: I also suggest not to use it; however, the main usage would be to use dotless full measure rests e.g. in 3/8 time (so R4. could look like r4 or r2). --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

Here are the answers I got from the publishers: Publisher 1 • Has no house rule for this situation. • But recommends to use a brevis rest for 1 measure in 4/2 time; for more measures use a “modern” horizontal beam rest (MMR.expand-limit = 1). • Alternatively do the same for 4/2 as for other time

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Patch**: review --> needs_work - **Comment**: So far I’ve got answer from all four German publishers I contacted but not from Faber Music (or even Elaine Gould). Three of these answers were very clear, I’ll wait for a clarification on the fourth and then summarize what they said and make a

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Patch**: countdown --> review - **Comment**: I can see from Rietveld that this is not yet quite ready for 'prime time', I will set this back to Review - while Malte gets his clarification. If nothing has changed by the next countdown, I'll put his back to 'Needs Work'. --- **

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

I am going to leave this on countdown as it looks like there is still some discussion. However if I have misinterpreted this, then it can be pushed. --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Patch**: push --> countdown --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Mon Jan 01, 2018 11:21 AM UTC **Owner:** Malte Meyn **Attachments:** -

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Patch**: review --> push - **Comment**: Patch on countdown for January 4th. --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Sun Dec 31, 2017 05:32 PM UTC **Owner:** Malte Meyn

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Needs**: --> - **Patch**: new --> review - **Type**: --> Enhancement - **Comment**: Passes make, make check and a full make doc. --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Sat Dec

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

move round-up-to-longer-rest to calc_closest_duration_log and improve regtest http://codereview.appspot.com/40043 --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Sat Dec 30, 2017 11:03 AM

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Patch**: review --> needs_work --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Wed Dec 27, 2017 04:02 PM UTC **Owner:** Malte Meyn **Attachments:** -

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **Needs**: --> - **Patch**: new --> review - **Type**: --> Enhancement - **Comment**: Passes make, make check and a ful make doc. --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Tue Dec

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

small whitespace correction http://codereview.appspot.com/40043 --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Tue Dec 26, 2017 09:41 PM UTC **Owner:** Malte Meyn **Attachments:** -

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

add regtest http://codereview.appspot.com/40043 --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Tue Dec 26, 2017 03:30 PM UTC **Owner:** Malte Meyn **Attachments:** -

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

This is the current situation (2.19.80) vs. the suggested patch, both use ``` \compressFullBarRests \override MultiMeasureRest.expand-limit = 15 \override MultiMeasureRest.usable-duration-logs = #'(2 1 0 -1 -2) ``` Attachments: -

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

use only whole rests and longer http://codereview.appspot.com/40043 --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Tue Dec 26, 2017 12:13 PM UTC **Owner:** Malte Meyn **Attachments:** -

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

- **assigned_to**: Malte Meyn - **Needs**: --> - **Type**: --> --- ** [issues:#3208] Wrong MultiMeasureRest glyph** **Status:** Started **Created:** Wed Feb 27, 2013 03:07 AM UTC by Anonymous **Last Updated:** Tue Dec 26, 2017 11:14 AM UTC **Owner:** Malte Meyn **Attachments:** -

[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #3208 Wrong MultiMeasureRest glyph

issue 3208: MMRs for > 1 m. only count m. MMRs with measure-count == 1 (single whole measure rests) don’t change their behaviour (i. e. they respect measure length when choosing the displayed rest symbol). MMRs with measure-count > 1 now only respect measure-count and don’t scale that by