Re: Gets vertical skylines from grob stencils (issue 5626052)

[Janek Warchoł]

On Tue, Mar 6, 2012 at 11:48 PM, Joe Neeman <address@hidden> wrote:
> 2012/3/6 <address@hidden>
>
>> Mike & all,
>>
>> i did a quick compile with patchset 36 and unfortunately didn't notice
>> significant speedups from previous version.
>
>
> Could you try the dev/jneem branch in git? It has some optimizations. If
> that doesn't help, could you please send me some of the worst files?

They look better indeed time-wise.
As for the output, i've only did a quick look and noticed this: why
you deleted horizontal padding?  This results in mf at the bottom
right of 1st page of Tota pulchra to jump into above lyrics.
Actually, small/zero horizontal padding might give better results in
some places, but that needs thorough investigation.

Compilation times:

PROCESSING  ../dynamics.ly  WITH MASTER: --------------

real    0m0.752s
user    0m0.632s
sys     0m0.096s

PROCESSING  ../dynamics.ly  WITH SKYLINES: ------------

real    0m1.002s
user    0m0.656s
sys     0m0.092s


PROCESSING  ../eja-mater/Eja-Mater.ly  WITH MASTER: --------------

real    0m7.195s
user    0m6.320s
sys     0m0.456s

PROCESSING  ../eja-mater/Eja-Mater.ly  WITH SKYLINES: ------------

real    0m10.792s
user    0m9.145s
sys     0m0.816s


PROCESSING  ../hairpins.ly  WITH MASTER: --------------

real    0m1.100s
user    0m0.772s
sys     0m0.072s

PROCESSING  ../hairpins.ly  WITH SKYLINES: ------------

real    0m0.866s
user    0m0.772s
sys     0m0.064s


PROCESSING  ../lyrics.ly  WITH MASTER: --------------

real    0m0.906s
user    0m0.752s
sys     0m0.084s

PROCESSING  ../lyrics.ly  WITH SKYLINES: ------------

real    0m0.955s
user    0m0.824s
sys     0m0.100s


PROCESSING  ../psalm94-reubke/psalm94.ly  WITH MASTER: --------------

real    0m46.639s
user    0m42.651s
sys     0m2.232s

PROCESSING  ../psalm94-reubke/psalm94.ly  WITH SKYLINES: ------------
^C
address@hidden ~$ ./skytest

PROCESSING  ../dynamics.ly  WITH MASTER: --------------

real    0m0.760s
user    0m0.652s
sys     0m0.076s

PROCESSING  ../dynamics.ly  WITH SKYLINES: ------------

real    0m0.807s
user    0m0.692s
sys     0m0.080s


PROCESSING  ../eja-mater/Eja-Mater.ly  WITH MASTER: --------------

real    0m7.295s
user    0m6.408s
sys     0m0.528s

PROCESSING  ../eja-mater/Eja-Mater.ly  WITH SKYLINES: ------------

real    0m10.482s
user    0m9.161s
sys     0m0.800s


PROCESSING  ../hairpins.ly  WITH MASTER: --------------

real    0m0.830s
user    0m0.736s
sys     0m0.064s

PROCESSING  ../hairpins.ly  WITH SKYLINES: ------------

real    0m0.811s
user    0m0.696s
sys     0m0.088s


PROCESSING  ../lyrics.ly  WITH MASTER: --------------

real    0m0.852s
user    0m0.704s
sys     0m0.116s

PROCESSING  ../lyrics.ly  WITH SKYLINES: ------------

real    0m0.968s
user    0m0.844s
sys     0m0.084s


PROCESSING  ../psalm94-reubke/psalm94.ly  WITH MASTER: --------------

real    0m45.944s
user    0m42.799s
sys     0m2.144s

PROCESSING  ../psalm94-reubke/psalm94.ly  WITH SKYLINES: ------------

real    1m9.338s
user    0m48.959s
sys     0m3.320s


PROCESSING  ../slanted-trillspan.ly  WITH MASTER: --------------
Unbound variable: ly:grob::vertical-skylines-from-stencil

real    0m2.852s
user    0m0.724s
sys     0m0.176s

PROCESSING  ../slanted-trillspan.ly  WITH SKYLINES: ------------

real    0m1.361s
user    0m0.800s
sys     0m0.128s


PROCESSING  ../tota-pulchra/tota-pulchra.ly  WITH MASTER: --------------

real    0m8.149s
user    0m7.416s
sys     0m0.360s

PROCESSING  ../tota-pulchra/tota-pulchra.ly  WITH SKYLINES: ------------

real    0m10.730s
user    0m9.625s
sys     0m0.672s

cheers,
Janek