Too much space allocated for minTieDistance when tie overlaps noteheads
Reported version
3.0
Priority
P1 - High
Type
Functional
Frequency
Once
Severity
S3 - Major
Reproducibility
Always
Status
closed
Regression
No
Workaround
No
Project
(version 3.0.2)
The change about tie spacing in MuseScore3 from in MuseScore2 is wrong.
The tie between each single note generates unnecessary space in MS3. You can find the tie is longer than the minimum tie length. MS2 doesn't generate such space in this case.
I think MuseScore should expand gaps only in the case the ties connect from chords to chords. MuseScore should achieve "Better Placement" in this picture.
See this topic.
https://musescore.org/en/comment/897419#comment-897419
Comments
Updated title to reflect what is actually happening. We add minTieDistance to the space between tied notes, but in many cases the tie will overlap the notehead so we don't need to allocate the full distance between the notes.
In 2.x I had implemented a fairly complex two-pass scheme to handle this, calculating the space during the first pass based on a whole bunch of factors then allocating it later. Ideally we could borrow that code (which is actually still present but has no effect), but at the very least we could at least optimize the single note case, which is the main problem here.
Turns out my original code to add space more intelligently was already reinstated, only a week after the current code to make room for ties was introduced! Left hand not knowing what the right was doing :-). Anyhow, we can now safely remove the new code, which is what is responsible for the extra space. Here is a PR to accomplish that:
https://github.com/musescore/MuseScore/pull/4732
Fixed in branch master, commit 1c8e61fb3c
fix #284794: too much space allocated for small ties
Fixed in branch master, commit dfbccdf15c
_Merge pull request #4732 from MarcSabatella/284794-tie-min
fix #284794: too much space allocated for small ties_
Fixed in branch 3.0.5, commit 23d3c530f2
_Merge pull request #4732 from MarcSabatella/284794-tie-min
fix #284794: too much space allocated for small ties_
Automatically closed -- issue fixed for 2 weeks with no activity.