Re: Enhance the -dpreview method for SVG output (issue 326960043 by address@hidden)

[Étienne Beaulé]

Hit send too soon...

With the fix, there's still the run-time error:
/home/ubuntu/lilypond/build/out/share/lilypond/current/scm/backend-library.scm:245:18:
In procedure ly:paper-book-systems in expression (ly:paper-book-systems
book):

/home/ubuntu/lilypond/build/out/share/lilypond/current/scm/backend-library.scm:245:18:
Wrong type argument in position 1 (expecting Paper_book): (#<Prob:
paper-system C++: Prob((Y-offset . 1.0) (system-grob . #<Grob System >)
(staff-refpoint-extent -9.77475935531354 . -9.77475935531354)
(last-in-score . #t) (page-turn-penalty) (page-break-penalty)
(page-turn-permission . allow) (page-break-permission . allow)
(vertical-skylines . #<Skyline_pair>) (stencil . #<Stencil>))() >

What would be the best way to approach this feature?

Thank you.
Étienne

2017-08-01 16:25 GMT-03:00 Étienne Beaulé <address@hidden>:

> Now that solves half of the issue. The other half is with the width, as my
> patch would (should) also adjust the width to fit the music, leaving no
> whitespace on any side.
>
> This feature is basically only available when using \includes "
> lilypond-book-preamble.ly" of which only works with the eps backend. My
> change would allow the use of "lilypond-book-preamble.ly" with the svg
> backend (which is already available in 'eps' so I wouldn't add it there
> too), but without the additional system by system output. Do you understand
> well now? I'm sorry if I wasn't clear.
>
> A bit embarrassing to have made a typo in a patch though, but it wouldn't
> work anyway. Maybe the way to go would be to implement crop-systems first
> for SVG. A worry for me is that it isn't lilypond "cropping" but
> GhostScript in the eps backend. Then, it would be worth continuing the plan
> of adding a variant of -dpreview.
>
> I'll be working more on this patch
>
> 2017-08-01 10:02 GMT-03:00 Paul <address@hidden>:
>
>> On 07/27/2017 03:21 AM, address@hidden wrote:
>>
>> This is a first attempt to merge the
>>> dump-page and dump-preview methods
>>> so that there is an option for
>>> cropping pages that are not just
>>> previews.
>>>
>>
>> I wonder... is the desired functionality already provided by
>> one-page-breaking and/or one-line-auto-height-breaking?
>>
>> http://lilypond.org/doc/v2.19/Documentation/notation/page-br
>> eaking#one_002dpage-page-breaking
>>
>> http://lilypond.org/doc/v2.19/Documentation/notation/page-br
>> eaking#one_002dline_002dauto_002dheight-page-breaking
>>
>> I haven't had time to look at the patch, and I'm still not sure exactly
>> what it's trying to do.
>>
>> -Paul
>>
>> _______________________________________________
>> lilypond-devel mailing list
>> address@hidden
>> https://lists.gnu.org/mailman/listinfo/lilypond-devel
>>
>
>