Re: 2.17.1 regtests

[address@hidden]

On 29 août 2012, at 18:58, Phil Holmes <address@hidden> wrote:

> I've just spent a happy few moments looking at around 900 regtest differences 
> - most of the ones between 2.17.0 and 17.1.  It's not a big deal except for 
> my sanity - we did know there would be a lot.  I've not checked them all, 
> because my PC gets rather hot while running the tests and it went sproing 
> towards the end, which sounded important so I aborted the tests at 
> tablature*.ly.  In practice, I think the PC is OK...
> 
> Anyway, I found a number that concern me.  See them at 
> http://www.philholmes.net/lilypond/regtestresults/2.17.1/.  Mike - some of 
> these will be halpful for your skylines work, so do please look at them. 
> There are others worth checking by those who value avoiding collisions. 
> Please do take a look.
> 
> There are also logfile diffs that may be worth considering.  Since Mike's 
> work allows systems to pack closer together, some of the checks on overflow 
> have changed, and it may be worth checking if the regtest is still doing its 
> job.  Logfile diffs are below.
> 
> HTH.
> 
> =======================================
> 
> Differences in 
> D:\Music\LilyPondDev\RegressionTests\2.17.0\regression\Output\Logfiles\page-breaking-end-of-score.log
> - warning: cannot fit music on page: overflow is 2.768012
> + warning: cannot fit music on page: overflow is 2.710000
> 
> Differences in 
> D:\Music\LilyPondDev\RegressionTests\2.17.0\regression\Output\Logfiles\page-breaking-min-systems-per-page2.log
> - warning: cannot fit music on page: overflow is 95.412213
> + warning: cannot fit music on page: overflow is 85.639124
> 
> Differences in 
> D:\Music\LilyPondDev\RegressionTests\2.17.0\regression\Output\Logfiles\page-breaking-page-count3.log
> - warning: cannot fit music on page: overflow is 6.881579
> + warning: cannot fit music on page: overflow is 2.479124
> 
> Differences in 
> D:\Music\LilyPondDev\RegressionTests\2.17.0\regression\Output\Logfiles\page-overflow-compression.log
> - warning: cannot fit music on page: overflow is 1.217463
> + Layout output to `page-overflow-compression.ps'...
> - warning: compressing music to fit
> + Converting to PNG...
> - Layout output to `page-overflow-compression.ps'...
> + Success: compilation successfully completed
> - Converting to PNG...
> +
> - Success: compilation successfully completed
> -
> 
> Filename 
> D:\Music\LilyPondDev\RegressionTests\2.17.0\regression\Output\safe.png does 
> not exist
> Filename 
> D:\Music\LilyPondDev\RegressionTests\2.17.0for2.17.1\regression\Output\safe.png
>  does not exist
> Differences in 
> D:\Music\LilyPondDev\RegressionTests\2.17.0\regression\Output\Logfiles\scheme-engraver.log
> - ) (length . #<Mom 1\8>))((class rest-event rhythmic-event music-event 
> StreamEvent) (length . #<Mom 1\8>) (duration . #<Duration 8 >) (origin . 
> #<location \scheme-engraver.ly:55:6
> + ) (length . #<Mom 1\8>))((class rest-event rhythmic-event music-event 
> StreamEvent) (length . #<Mom 1\8>) (duration . #<Duration 8 >) (origin . 
> #<location \schem
> - Converting to PNG...>)) >
> + Converting to PNG...e-engraver.ly:55:6>)) >
> 
> Differences in 
> D:\Music\LilyPondDev\RegressionTests\2.17.0\regression\Output\Logfiles\skiptypesetting-all-true.log
> - Success: compilation successfully completed
> + warning: 6 expected warning(s) not encountered:
> -
> +         didn't find a vertical alignment in this system
> 
> Differences in 
> D:\Music\LilyPondDev\RegressionTests\2.17.0\regression\Output\Logfiles\stem-length-estimation.log
> - Layout output to `stem-length-estimation.ps'...
> + warning: cannot fit music on page: overflow is 8.901118
> - Converting to PNG...
> + warning: compressing music to fit
> - Success: compilation successfully completed
> + Layout output to `stem-length-estimation.ps'...
> -
> + Converting to PNG...
> 
> 
> --
> Phil Holmes
> 
> 
> 
> _______________________________________________
> lilypond-devel mailing list
> address@hidden
> https://lists.gnu.org/mailman/listinfo/lilypond-devel

Thanks Phil!

The regtest that worries me most is les-nereides.ly.

There is a comment starting on line 827 of axis-group-engraver.cc from Joe 
(vintage 2008) that describes this scenario. It's just that excess spacing has 
always hid it.  It is now the time to tackle it head on, as w/ the new skyline 
patch this type of scenario will come up more often.

Cheers,
MS