Re: Persian music package

[Kees van den Doel]

On Sun, Aug 1, 2021 at 10:40 AM Kees van den Doel <kvdoel@gmail.com> wrote:

On Sun, Aug 1, 2021 at 2:29 AM Thomas Morley <thomasmorley65@gmail.com> wrote:

Furthermore I'd replace the override for Accidental.extra-offset by:
    \override Accidental.Y-offset =

Typo I assume and you mean "\override Accidental.extra-offset ="?

 

      #(lambda (grob)
          (cdr
            (assoc-get
              (ly:grob-property grob 'alteration)
              persianStringsOffsets)))

That doesn't work because now you take the cdr twice. I think (assoc-get ... is the same as (cdr (assoc .. isn't it?

But assoc-get without cdr is nicer of course, didn't know about that.

No that doesn't work, I must not understand something. Can you explain why you suggest the change? If I keep the cdr as in your example it runs but  offsets are no longer right.  It works fine with assoc.

Cheers,

Kees