weird attachment of Lyrics to NullVoice in polyphony

Issue Tracker for LilyPond

Brought to you by: fedelibre, hahnjo, philholmes, tdanielsmusic

#3825 weird attachment of Lyrics to NullVoice in polyphony

Status: Verified

Owner: nobody

Labels: fixed_2_19_13 (3)

Needs:

Patch:

Type: Defect

Updated: 2014-09-04

Created: 2014-01-27

Creator: Anonymous

Private: No

Originally created by: *anonymous

Originally created by: Elu...@gmail.com
Originally owned by: k-ohara5...@oco.net

with this source code - which is a shortened version of http://lilypond.1069038.n5.nabble.com/Weird-behaviour-of-new-NullVoice-feature-td158448.html by Yannick

<<
\new Staff <<
    \new Voice = "alto" \relative c' { \voiceTwo c4 b c2 | }
    \new NullVoice = "xyz" \relative c' { c4 b c2 | }
    \new Voice = "melodie" \relative c''{ \voiceOne c4. g8 c2 | }
>>
\lyricsto "xyz" \new Lyrics \lyricmode {Glo ri a }
>>
\version "2.19.1"

the lyrics output doesn't follow the voice:

1 Attachments

test15.png

Discussion

Google Importer - 2014-07-21

Originally posted by: k-ohara5...@oco.net

http://codereview.appspot.com/117050043

Labels: patch-new
Owner: k-ohara5...@oco.net

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-22

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests.

Labels: -patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-24

Originally posted by: pkx1...@gmail.com

Patch on countdown for July 27th 2014

Labels: -Patch-review Patch-countdown

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-26

Originally posted by: pkx1...@gmail.com

Patch counted down - please push (although Janek said he would take a closer look 'today' - Rietveld comment). So do with it as you will.

Labels: -Patch-countdown Patch-push
Cc: janek.li...@gmail.com

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-27

Originally posted by: janek.li...@gmail.com

(No comment was entered for this change.)

Blocking: lilypond:3834

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-27

Originally posted by: janek.li...@gmail.com

(No comment was entered for this change.)

Blocking: lilypond:3734

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-27

Originally posted by: k-ohara5...@oco.net

(No comment was entered for this change.)

Labels: -Patch-push Patch-new
Status: Started

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-28

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes 'make', 'make check' and a full 'make doc'. Includes a full make doc

Labels: -Patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-28

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes 'make', 'make check' and a full 'make doc'. Includes a full make doc

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-28

Originally posted by: k-ohara5...@oco.net

For this one bug, NullVoice needs the Grob_pq_engraver

I'm pondering suggestions on the tracker.

Labels: -Patch-review
Status: Accepted

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-07-31

Originally posted by: janek.li...@gmail.com

Hi Keith,
sorry for a possibly silly question, but I'm not sure what's the status of this issue. As far as i can see, the latest version of the patch works quite well. I understand that we would like to have a new melisma_signal_engraver to decouple handling of melismas from ties and slurs; however, why not push this patch as-is and open another issue for "extracting" melisma_signal_engraver?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-01

Originally posted by: k-ohara5...@oco.net

I was thinking of making it general, so people can put NullVoice into a Staff, but I am unlikely to get around to that.

Labels: patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-04

Originally posted by: pkx1...@gmail.com

Patch on countdown of August 8th 2014

Labels: -patch-review Patch-countdown

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-08

Originally posted by: pkx1...@gmail.com

Patch counted down - please push

Labels: -Patch-countdown Patch-push

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-10

Originally posted by: k-ohara5...@oco.net

I didn't get onto Linux this weekend to push, and now there is another vote to make NullVoice more general, so it can be put into a Staff.

Labels: -Patch-push Patch-needs_work

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-23

Originally posted by: k-ohara5...@oco.net

(No comment was entered for this change.)

Labels: -Patch-needs_work Patch-new

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-23

Originally posted by: pkx1...@gmail.com

Patchy the autobot says: passes tests. Includes a full make doc

Labels: -Patch-new Patch-review

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-24

Originally posted by: pkx1...@gmail.com

Patch on countdown for August 27th

Labels: -Patch-review Patch-countdown

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-26

Originally posted by: pkx1...@gmail.com

Patch counted down - please push

Labels: -Patch-countdown Patch-push

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-29

Originally posted by: k-ohara5...@oco.net

Original example should have the 'ri' syllable under the B([r958d1e069079380c7a6cdc118f6b5b4e4a88239e])

Labels: -Patch-push
Status: Fixed2_19_13

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-08-29

Originally posted by: k-ohara5...@oco.net

(No comment was entered for this change.)

Labels: fixed_2_19_13
Status: Fixed

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Google Importer - 2014-09-04

Originally posted by: ColinPKC...@gmail.com

(No comment was entered for this change.)

Status: Verified

If you would like to refer to this comment somewhere else in this project, copy and paste the following link: