Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread dak


https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode465
Documentation/learning/fundamental.itely:465: optionally followed by one
or more post-events.  Post-events add
On 2018/04/30 22:19:12, Carl wrote:

On 2018/04/30 21:49:37, thomasmorley651 wrote:
> I'd completely delete 'post-event'.
> From a musical thinking it makes no sense. An articulation is not

performed

> _after_ the note.
> To explain it programmatical, this is not the right place, imho.
>
> Why not simply:
>
> "A note entry in LilyPond consists of a pitch, followed by a

duration,

> optionally followed by things such as articulations, fingerings,

string

numbers,
> slurs, ties, explanatory text, etc."



We could do this.  But ultimately all the things that attach to notes

like this

are called post-events in the internals reference.  So I don't think

it's a bad

idea to introduce the LilyPond term here, just like we do for pitch

and

duration.  All three terms (pitch, duration, post-event) are LilyPond

terms, not

musical terms.



We aren't explaining music in this section.  We are explaining

LilyPond

constructs.


"pitch" and "duration" are certainly valid musical terms outside of
LilyPond and describe musical entities even within LilyPond.
"post-event" does not carry meaning separate from the LilyPond grammar.

It might make sense to introduce it as a LilyPond term by putting it in
quote marks first mention:

one or more @q{post-events}.  Post-events are placed after a main event
in the input and constitute things such as ...

https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode481
Documentation/learning/fundamental.itely:481: Post-events follow the
note to which they are attached.  Suppose we
On 2018/04/30 22:19:12, Carl wrote:

I don't think I agree that things are attached with "-" signs.


It's more a less a matter of what one wants to declare the rule and what
the exception.  It's more of a semantic difference, so it's more the
question of which view might be more helpful.

I'd lean towards keeping your version here: I think it better matches
what we use elsewhere.

https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread Carl . D . Sorensen

According to parser.yy:

In line 3259, a post_event is either:

1) post_event_nofinger, or
2) '-' plus a fingering

In line 3200, a post_event_nofinger is either

1) direction_less_event
2) script_dir plus a music_function
3,4 ) Lyric hyphen or lyric extender (Not relevant to this discussion).
5) script_dir plus a direction_reqd_event
6) script_dir plus a direction_less_event
7) '^' plus fingering
8) '_' plus fingering


In line 3275, a direction_less_event is either
1) string_number_event
2) tremolo
3)event_function_event

In line 3288, a direction_reqd_event is either
1) gen_text_def
2) script_abbreviation

In line 3387, a gen_text_def is either
1) full_markup
2) STRING
3) SYMBOL
4) embedded_scm

In line 3413, a fingering is
1) UNSIGNED

The way I read this, NR 5.4.2 is exactly right.  A direction indicator
is required for
1) markups (gen_text_def case 1)
2) strings (gen_text_def case 2)
3) abbreviated forms of articulations (script_abbreviation)
4) fingerings (post_event case 2, direction_less_event case 7,
direction_less_event case 8)
5) \tweak or \tag (I'm not sure about this, but I think it's
gen_text_def case 4)

Given this, I don't think we should say that '-' is what we use to
insert post_events.

I am of the opinion that we should not include '-' as a specific element
in this discussion.  I think it should say as-is.

Thanks,

Carl



https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread thomasmorley65

On 2018/04/30 22:55:35, thomasmorley651 wrote:

On 2018/04/30 22:19:12, Carl wrote:



> I don't think I agree that things are attached with "-" signs.  For

example,

> \staccato, \mordent, \turn, \fermata, \prall, (, [. None of these

are attached

> with "-" signs, although they can have a direction indicator (-, _ ,

^)

> preceding them if desired.  At least, that is what the N.R. 5.4.2

says.


Here the NR is not entirely complete.
"_" and "^" _are_ direction-modifiers, ofcourse.
"-" _is_ the method to insert a post-event into a list of other

post-events.

For convenience it can be omitted, if no ambiguity is present. (It

can't be

omitted before a fingering, for example)
The direction of this post-event is the default-direction for said

post-event.

But this more a side-effect.



Also, see:
{ \displayLilyMusic c'\tenuto }
=> c'4--


Well, I had a look in parser.yy. Although, I can't say I really
understand it, I'm not longer sure above is true.
Maybe David K can say more.




https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread thomasmorley65

On 2018/04/30 22:19:12, Carl wrote:


https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely

File Documentation/learning/fundamental.itely (right):



https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode465

Documentation/learning/fundamental.itely:465: optionally followed by

one or more

post-events.  Post-events add
On 2018/04/30 21:49:37, thomasmorley651 wrote:
> I'd completely delete 'post-event'.
> From a musical thinking it makes no sense. An articulation is not

performed

> _after_ the note.
> To explain it programmatical, this is not the right place, imho.
>
> Why not simply:
>
> "A note entry in LilyPond consists of a pitch, followed by a

duration,

> optionally followed by things such as articulations, fingerings,

string

numbers,
> slurs, ties, explanatory text, etc."



We could do this.  But ultimately all the things that attach to notes

like this

are called post-events in the internals reference.  So I don't think

it's a bad

idea to introduce the LilyPond term here, just like we do for pitch

and

duration.  All three terms (pitch, duration, post-event) are LilyPond

terms, not

musical terms.


Pitch and duration are surely terms every musician and every
music-typesetter will know, they are _musical_ terms as well.
Not so post-event.
Maybe it would be sufficient to reword:
"optionally followed by one or more elements, which are called
"post-events".



https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode481

Documentation/learning/fundamental.itely:481: Post-events follow the

note to

which they are attached.  Suppose we
On 2018/04/30 21:49:37, thomasmorley651 wrote:
>
> Here as well.
>
> Also, I think it's important to drop a sentence about the "-"-signs,

which

> actually attach those optional elements to the note.
>
> So my suggestion:
>
> "Optional elements are added at the end of the initial

note-duration-entry.

> Probably using a "-"-sign, which can be omitted, if no ambiguity

occurs.

> Suppose we ..."
>
>



I don't think I agree that things are attached with "-" signs.  For

example,

\staccato, \mordent, \turn, \fermata, \prall, (, [. None of these are

attached

with "-" signs, although they can have a direction indicator (-, _ ,

^)

preceding them if desired.  At least, that is what the N.R. 5.4.2

says.

Here the NR is not entirely complete.
"_" and "^" _are_ direction-modifiers, ofcourse.
"-" _is_ the method to insert a post-event into a list of other
post-events.
For convenience it can be omitted, if no ambiguity is present. (It can't
be omitted before a fingering, for example)
The direction of this post-event is the default-direction for said
post-event. But this more a side-effect.

Also, see:
{ \displayLilyMusic c'\tenuto }
=> c'4--



If we want to talk about direction indicators here, I think we can

give a brief

introduction.  If not, I think we should leave them out completely.


Agreed, direction-modifiers shouldn't be explained here in the
fundamentals.


In the LM
and the NR, the direction indicators are always included when we add

the

post-events, if they are needed.



https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode488

Documentation/learning/fundamental.itely:488: {c'8-1--(~^\markup{"text
annotation"} c' d')}
On 2018/04/30 21:49:37, thomasmorley651 wrote:
>
> For the sake of simplicity I'd not use direction-modifiers and enter

the text

> without explicit \markup, i.e.:
> {c'8-1--(~-"text annotation" c' d')}



I think it's actually nicer not to have so many "-" characters; they

make it

confusing, in my opinion.


Ofcourse, but I don't understand the relevance here.
{c'8-1--(~-"text annotation" c' d')} does not use more "-" than the
other code.

Btw, maybe better to add a space after { and before }.




https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread Carl . D . Sorensen

Harm,

Thanks for the input.  I'm not sure I agree with you on all this, but
I'm certainly open to being convinced.  I've got specific replies to
your inline comments.


https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode465
Documentation/learning/fundamental.itely:465: optionally followed by one
or more post-events.  Post-events add
On 2018/04/30 21:49:37, thomasmorley651 wrote:

I'd completely delete 'post-event'.
 From a musical thinking it makes no sense. An articulation is not

performed

_after_ the note.
To explain it programmatical, this is not the right place, imho.



Why not simply:



"A note entry in LilyPond consists of a pitch, followed by a duration,
optionally followed by things such as articulations, fingerings,

string numbers,

slurs, ties, explanatory text, etc."


We could do this.  But ultimately all the things that attach to notes
like this are called post-events in the internals reference.  So I don't
think it's a bad idea to introduce the LilyPond term here, just like we
do for pitch and duration.  All three terms (pitch, duration,
post-event) are LilyPond terms, not musical terms.

We aren't explaining music in this section.  We are explaining LilyPond
constructs.

https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode481
Documentation/learning/fundamental.itely:481: Post-events follow the
note to which they are attached.  Suppose we
On 2018/04/30 21:49:37, thomasmorley651 wrote:


Here as well.



Also, I think it's important to drop a sentence about the "-"-signs,

which

actually attach those optional elements to the note.



So my suggestion:



"Optional elements are added at the end of the initial

note-duration-entry.

Probably using a "-"-sign, which can be omitted, if no ambiguity

occurs.

Suppose we ..."




I don't think I agree that things are attached with "-" signs.  For
example, \staccato, \mordent, \turn, \fermata, \prall, (, [. None of
these are attached with "-" signs, although they can have a direction
indicator (-, _ , ^) preceding them if desired.  At least, that is what
the N.R. 5.4.2 says.

If we want to talk about direction indicators here, I think we can give
a brief introduction.  If not, I think we should leave them out
completely.  In the LM and the NR, the direction indicators are always
included when we add the post-events, if they are needed.

https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode488
Documentation/learning/fundamental.itely:488: {c'8-1--(~^\markup{"text
annotation"} c' d')}
On 2018/04/30 21:49:37, thomasmorley651 wrote:


For the sake of simplicity I'd not use direction-modifiers and enter

the text

without explicit \markup, i.e.:
{c'8-1--(~-"text annotation" c' d')}


I think it's actually nicer not to have so many "-" characters; they
make it confusing, in my opinion.

https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread thomasmorley65

Hi Carl,

many thanks for your work.
Some remarks:


https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode465
Documentation/learning/fundamental.itely:465: optionally followed by one
or more post-events.  Post-events add
I'd completely delete 'post-event'.
From a musical thinking it makes no sense. An articulation is not
performed _after_ the note.
To explain it programmatical, this is not the right place, imho.

Why not simply:

"A note entry in LilyPond consists of a pitch, followed by a duration,
optionally followed by things such as articulations, fingerings, string
numbers, slurs, ties, explanatory text, etc."

https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode481
Documentation/learning/fundamental.itely:481: Post-events follow the
note to which they are attached.  Suppose we

Here as well.

Also, I think it's important to drop a sentence about the "-"-signs,
which actually attach those optional elements to the note.

So my suggestion:

"Optional elements are added at the end of the initial
note-duration-entry.  Probably using a "-"-sign, which can be omitted,
if no ambiguity occurs.
Suppose we ..."

https://codereview.appspot.com/343060043/diff/40001/Documentation/learning/fundamental.itely#newcode488
Documentation/learning/fundamental.itely:488: {c'8-1--(~^\markup{"text
annotation"} c' d')}

For the sake of simplicity I'd not use direction-modifiers and enter the
text without explicit \markup, i.e.:
{c'8-1--(~-"text annotation" c' d')}

https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread Carl Sorensen


On 4/30/18, 11:04 AM, "lilypond-devel on behalf of 
tdanielsmu...@googlemail.com" 
 wrote:

@ref definitely should work.  It may be that the heading for one
of them is "Articulation and dynamics", not "Articulations and
dynamics" - no "s".  Using @ruser bypasses the check, I think.

OK, that's clearly the issue.

Now, how should it be fixed?  Should we stick with "Articulation and dynamics", 
even though the next menu down is "Articulations"?  Or should we change to 
"Articulations and dynamics"?  I can do either one.

Thanks,
Carl
 

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re[2]: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread Trevor



Carl, you wrote 30/04/2018 18:16:05


On 4/30/18, 11:04 AM, "lilypond-devel on behalf of 
tdanielsmu...@googlemail.com" 
 wrote:


   @ref definitely should work.  It may be that the heading for one
   of them is "Articulation and dynamics", not "Articulations and
   dynamics" - no "s".  Using @ruser bypasses the check, I think.

OK, that's clearly the issue.

Now, how should it be fixed?  Should we stick with "Articulation and 
dynamics", even though the next menu down is "Articulations"?  Or 
should we change to "Articulations and dynamics"?  I can do either one.
Well, I'd prefer adding the "s" to the title, as long as it doesn't 
break other references (I don't know if there are any, but if there are 
any in the LM they should show up as errors in a make doc.)  
Documentation/scripts/auxiliar/ref_check.py should check all refs for 
you, if it still works - it's a while since I used it.


Trevor


___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread tdanielsmusic


https://codereview.appspot.com/343060043/diff/20001/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/343060043/diff/20001/Documentation/learning/fundamental.itely#newcode495
Documentation/learning/fundamental.itely:495: @rlearning{Adding text}.
On 2018/04/30 15:51:16, Carl wrote:

On 2018/04/30 15:47:26, Trevor Daniels wrote:
> Not sure if it makes any difference but we usually use
> @ref for references within the same manual.



Fundamentals is in a different source file than common notation.  If I

use @ref,

it breaks the info build.  So I had to use @rlearning to get it to

pass make

doc.



If I'm doing something wrong, and there's some way to get @ref to

work, I'd be

happy to know about it.


@ref definitely should work.  It may be that the heading for one
of them is "Articulation and dynamics", not "Articulations and
dynamics" - no "s".  Using @ruser bypasses the check, I think.

https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread Carl . D . Sorensen

Thanks for the feedback, Trevor.


https://codereview.appspot.com/343060043/diff/20001/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/343060043/diff/20001/Documentation/learning/fundamental.itely#newcode495
Documentation/learning/fundamental.itely:495: @rlearning{Adding text}.
On 2018/04/30 15:47:26, Trevor Daniels wrote:

Not sure if it makes any difference but we usually use
@ref for references within the same manual.


Fundamentals is in a different source file than common notation.  If I
use @ref, it breaks the info build.  So I had to use @rlearning to get
it to pass make doc.

If I'm doing something wrong, and there's some way to get @ref to work,
I'd be happy to know about it.

https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread tdanielsmusic

Hi Carl

Another minor nit!  Otherwise looks fine.

Trevor



https://codereview.appspot.com/343060043/diff/20001/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/343060043/diff/20001/Documentation/learning/fundamental.itely#newcode495
Documentation/learning/fundamental.itely:495: @rlearning{Adding text}.
Not sure if it makes any difference but we usually use
@ref for references within the same manual.

https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread Carl . D . Sorensen

On 2018/04/30 12:08:20, Trevor Daniels wrote:



https://codereview.appspot.com/343060043/diff/1/Documentation/learning/fundamental.itely#newcode488

Documentation/learning/fundamental.itely:488: @end lilypond
I'd prefer \relative or nothing in this example. \absolute
has not been introduced earlier, in fact it's not used at
all in the LM.


Good idea.  I went with nothing.



https://codereview.appspot.com/343060043/diff/1/Documentation/learning/fundamental.itely#newcode492

Documentation/learning/fundamental.itely:492: @ruser{Expressive

marks},

Might be good to point to the relevant sections in the LM
as well, in Section 2.1


Also a good idea.  I've added the references.

Thanks,

Carl



https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread Carl . D . Sorensen

Reviewers: Be-3, Trevor Daniels,

Message:
On 2018/04/30 11:37:09, Be-3 wrote:

Hi Carl,



Concise, comprehensible, - LGTM!



Perhaps it should be explicitly pointed out that the duration

shorthand does not

work for rests.
There have been some misconceptions lately on the user list, and so I

think this

detail probably deserves special attendance.


I've added a sentence that should clarify.


https://codereview.appspot.com/343060043/diff/1/Documentation/learning/fundamental.itely#newcode487

Documentation/learning/fundamental.itely:487: \absolute

{c'8-1--(~^\markup{"text

annotationi"} c' d')}
Shouldn't it be "text annotation"?

Yes.  Now you know which editor I was using to create the patch -- it's
vim leftovers.

Thanks for the catch.



Description:
Clarify notation for slurs and beams

Note that the opening code for slurs and beams comes after
the first note of the slur or beam.

Also, add a section about notes containing pitches, durations,
and post-events.

Please review this at https://codereview.appspot.com/343060043/

Affected files (+41, -2 lines):
  M Documentation/learning/common-notation.itely
  M Documentation/learning/fundamental.itely


Index: Documentation/learning/common-notation.itely
diff --git a/Documentation/learning/common-notation.itely  
b/Documentation/learning/common-notation.itely
index  
b9b62cf3b321452ec117290c7f9b68cf4246e39b..a6136a421a0b84f854d793a060e17b8551ee5eb6  
100644

--- a/Documentation/learning/common-notation.itely
+++ b/Documentation/learning/common-notation.itely
@@ -275,6 +275,7 @@ Notation Reference:
 @funindex ( ... )
 @funindex \( ... \)

+
 @menu
 * Ties::
 * Slurs::
@@ -318,7 +319,8 @@ Music Glossary: @rglos{slur}.

 A @notation{slur} is a curve drawn across many notes.  The
 starting note and ending note are marked with @code{(} and
-@code{)} respectively.
+@code{)} respectively. Note that the @code{(} comes after
+the first note of the slur.

 @lilypond[verbatim,quote]
 \relative { d''4( c16) cis( d e c cis d) e( d4) }
@@ -369,6 +371,9 @@ Notation Reference:
 @node Articulation and dynamics
 @subsection Articulation and dynamics

+Articulations and dynamics are indicated by adding special codes
+after the notes to which they apply,
+
 @menu
 * Articulations::
 * Fingerings::
@@ -526,6 +531,7 @@ All @notation{beams} are drawn automatically:
 If you do not like the automatic beams, they may be overridden
 manually.  To correct just an occasional beam mark the first note
 to be beamed with @code{[} and the last one with @code{]}.
+Note that @code{[} comes after the first beamed note.

 @lilypond[verbatim,quote]
 \relative { a'8[ ais] d[ ees r d] c16 b a8 }
@@ -597,7 +603,7 @@ Music Glossary: @rglos{note value}, @rglos{triplet}.

 @notation{Tuplets} are made with the @code{\tuplet} keyword.  It
 takes two arguments: a fraction and a piece of music.  The
-fraction is the number of tuplet notes over the number
+fraction is the number of tuplet notes over the number
 of notes normally filling the same duration.
 For triplets, there are three notes instead of two, so
 @notation{triplets} have 3/2 as their fraction.
Index: Documentation/learning/fundamental.itely
diff --git a/Documentation/learning/fundamental.itely  
b/Documentation/learning/fundamental.itely
index  
40efc3777ab567d7f440a360839e4a3d1570fa25..fc8290a14314c2b916393e414373aad305a2e301  
100644

--- a/Documentation/learning/fundamental.itely
+++ b/Documentation/learning/fundamental.itely
@@ -40,6 +40,7 @@ description of the input format, see @ruser{File  
structure}.

 * Introduction to the LilyPond file structure::
 * Score is a (single) compound musical expression::
 * Nesting music expressions::
+* Structure of a note entry::
 * On the un-nestedness of brackets and ties::
 @end menu

@@ -457,6 +458,38 @@ These require further commands which
 have not yet been introduced.  See @ref{Size of objects},
 and @ruser{Ossia staves}.

+@node Structure of a note entry
+@subsection Structure of a note entry
+
+A note entry in LilyPond consists of a pitch, followed by a duration,
+optionally followed by one or more post-events.  Post-events add
+things such as articulations, fingerings, string numbers, slurs, ties
+and explanatory text.
+
+The pitch may be explicitly defined using the current LilyPond input
+language as described in @ruser{Note names in other languages}.  The
+pitch may be omitted.  If the pitch is omitted, the pitch of a current
+note will be the same as the pitch of the previous note in the input
+file, see @ruser{Durations}.
+
+The duration includes a number and optionally one or more dots.  If a
+duration is not explicitly defined, the duration of a current note will
+be the same as the duration of the previous note, chord, rest, or spacer
+rest, see @ruser{Durations}.
+
+Post-events follow the note to which they are attached.  Suppose we
+want to have an eighth note c' with a fingering of 1, a tenuto
+articulation, a slur beginning with the note, a tie beginning with
+the note, 

Re: Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread tdanielsmusic

Hi Carl

LGTM, with a couple of minor comments.

Trevor



https://codereview.appspot.com/343060043/diff/1/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/343060043/diff/1/Documentation/learning/fundamental.itely#newcode488
Documentation/learning/fundamental.itely:488: @end lilypond
I'd prefer \relative or nothing in this example. \absolute
has not been introduced earlier, in fact it's not used at
all in the LM.

https://codereview.appspot.com/343060043/diff/1/Documentation/learning/fundamental.itely#newcode492
Documentation/learning/fundamental.itely:492: @ruser{Expressive marks},
Might be good to point to the relevant sections in the LM
as well, in Section 2.1

https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


PATCHES - Countdown for April 30th

2018-04-30 Thread James Lowe
Hello,

Here is the current patch countdown list. The next countdown will be on May 3rd.

A quick synopsis of all patches currently in the review process can be found 
here:

http://philholmes.net/lilypond/allura/



Push:

5312 Key cancellation glyph position inconsistent - Torsten Hammerle
https://sourceforge.net/p/testlilyissues/issues/5312
http://codereview.appspot.com/343020043

Countdown: No patches on countdown at this time

Review: No patches in review at this time.

New:

5316 Clarify notation for slurs and beams - Carl Sorensen
https://sourceforge.net/p/testlilyissues/issues/5316
http://codereview.appspot.com/343060043

Regards

James
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Clarify notation for slurs and beams (issue 343060043 by carl.d.soren...@gmail.com)

2018-04-30 Thread torsten . haemmerle

Hi Carl,

Concise, comprehensible, - LGTM!

Perhaps it should be explicitly pointed out that the duration shorthand
does not work for rests.
There have been some misconceptions lately on the user list, and so I
think this detail probably deserves special attendance.

Thanks,
Torsten


https://codereview.appspot.com/343060043/diff/1/Documentation/learning/fundamental.itely
File Documentation/learning/fundamental.itely (right):

https://codereview.appspot.com/343060043/diff/1/Documentation/learning/fundamental.itely#newcode487
Documentation/learning/fundamental.itely:487: \absolute
{c'8-1--(~^\markup{"text annotationi"} c' d')}
Shouldn't it be "text annotation"?

https://codereview.appspot.com/343060043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel