Re: help with bash script to translate @ref{} items in translated manuals

[Federico Bruni]

Hi Andrew

Thank you, but it hangs forever and nothing happens.


Il giorno sab 22 apr 2017 alle 5:37, Andrew Bernard 
 ha scritto:

Hi Federico,

I believe you are trying to automate a set of translations, correct? 
If so, here's a way to do it in perl that avoids all the shell 
convolutions. I assume you do know perl. If not, always worth knowing 
for this sort of quick work.


Just add the translations to the hash table in the script. The virtue 
of this small tool is that it will tell you if you missed any.


Reads file from stdin. Outputs to stdout. You can figure how to 
process the whole directory. There's a hundred ways to do this. Some 
people like to open all the files in the perl script. I prefer to 
keep it simple.


Hope this may be useful for now and in the future.


Andrew

== snip

#!/usr/bin/perl

use strict;
use warnings;

my @ref;

# translation table
my %translations = (
'Automatic beams' => 'Automatic Beams in Italian',
'Stems' => 'Stems in Italian',
);

while (<>) {
if (@ref = /\@ref\{(.+)\}/) {
if (exists $translations{$ref[0]}) {
s/$ref[0]/$translations{$ref[0]}/;
}
else {
print STDERR "no translation for $ref[0]", "\n";
}
}
print;
}

== snip




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

Re: Improve elbowed-hairpin (issue 324800043 by thomasmorle...@gmail.com)

[thomasmorley65]

On 2017/04/24 14:42:18, david.nalesnik wrote:

LGTM.



I've pointed out two minor issues, but I don't believe they should

hold up the

review process.



By the way, applying



\override Hairpin.style = #'dashed-line to the regtest
input/regression/ferneyhough-hairpins.ly looks great!



https://codereview.appspot.com/324800043/diff/1/scm/output-lib.scm
File scm/output-lib.scm (right):



https://codereview.appspot.com/324800043/diff/1/scm/output-lib.scm#newcode1171

scm/output-lib.scm:1171: (if (not (pair? (cdr pts)))
Note that this will fail if pts is '().  Perhaps you should check the

length of

pts in elbowed-hairpin -- there must be at least two pairs in the pts

alist for

a viable line.


Yep.

I decided to let the condition for the inner 'connected-points' in
place, but to
emit a meaningful warning if it is called in 'make-connected-line' with
less than
two points. In this case an empty-stencil is returned.

Seems cleaner to do it early than later in 'elbowed-hairpin'.


https://codereview.appspot.com/324800043/diff/1/scm/output-lib.scm#newcode1172

scm/output-lib.scm:1172: (apply ly:stencil-add empty-stencil ls)
Why not use reduce here.


Done.

This warrants a new patchy-run because new functionality is added.



https://codereview.appspot.com/324800043/

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

Slashed Half-flat and add. scales for Arabic music (issue 317550043 by amir...@hotmail.de)

[dak]

https://codereview.appspot.com/317550043/diff/1/ly/arabic.ly
File ly/arabic.ly (right):

https://codereview.appspot.com/317550043/diff/1/ly/arabic.ly#newcode28
ly/arabic.ly:28:
Whitespace error?

https://codereview.appspot.com/317550043/diff/1/ly/arabic.ly#newcode86
ly/arabic.ly:86:
Whitespace error?

https://codereview.appspot.com/317550043/diff/1/ly/arabic.ly#newcode88
ly/arabic.ly:88:
Whitespace error?

Maybe check with

git log --check

or so?

https://codereview.appspot.com/317550043/

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

Re: Improve elbowed-hairpin (issue 324800043 by thomasmorle...@gmail.com)

[david . nalesnik]

LGTM.

I've pointed out two minor issues, but I don't believe they should hold
up the review process.

By the way, applying

\override Hairpin.style = #'dashed-line to the regtest
input/regression/ferneyhough-hairpins.ly looks great!


https://codereview.appspot.com/324800043/diff/1/scm/output-lib.scm
File scm/output-lib.scm (right):

https://codereview.appspot.com/324800043/diff/1/scm/output-lib.scm#newcode1171
scm/output-lib.scm:1171: (if (not (pair? (cdr pts)))
Note that this will fail if pts is '().  Perhaps you should check the
length of pts in elbowed-hairpin -- there must be at least two pairs in
the pts alist for a viable line.

https://codereview.appspot.com/324800043/diff/1/scm/output-lib.scm#newcode1172
scm/output-lib.scm:1172: (apply ly:stencil-add empty-stencil ls)
Why not use reduce here.

https://codereview.appspot.com/324800043/

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

Re: PATCHES - Countdown for April 24th

[Thomas Morley]

2017-04-24 10:00 GMT+02:00 James :
> Hello,
>
> Here is the current patch countdown list. The next countdown will be on
> April 27th.
>
> A quick synopsis of all patches currently in the review process can be
> found here:
>
> http://philholmes.net/lilypond/allura/
>
> 
>
>
>
> Push:
>
>
> 5122 Fix not scaling stem in note-by-number-markup - Thomas Morley
> https://sourceforge.net/p/testlilyissues/issues/5122
> http://codereview.appspot.com/324780043


I've pushed the patch to staging as:

commit eee677c480c78d58a5215e246575aa94ba2d1897
Author: Thomas Morley 
Date:   Sun Apr 16 17:11:05 2017 +0200

Issue 5122 Fix not scaling stem in note-by-number-markup

Done by calling text-font-size from layout, which defaults to 11.
Then scaling stem-thickness and stem-length by division of
text-font-size through 11.
This ensures a stem-appearance according to the text-properties of
MetronomeMark.

Though, I can't write anything at sourceforge, atm.
No clue why, will try again later.

Cheers,
  Harm

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

PATCHES - Countdown for April 24th

[James]

Hello,

Here is the current patch countdown list. The next countdown will be on
April 27th.

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

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





Push:


5122 Fix not scaling stem in note-by-number-markup - Thomas Morley
https://sourceforge.net/p/testlilyissues/issues/5122
http://codereview.appspot.com/324780043


Countdown: No patches in Countdown at this time.


Review:


5123 Improve elbowed-hairpin - Thomas Morley
https://sourceforge.net/p/testlilyissues/issues/5123
http://codereview.appspot.com/324800043


New: No New patches at this time.



Regards

James

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