good news! created a successful spanning engraver entirely on the scheme end.

;ve uploaded to Reitveld and are waiting review. As soon as they get commented upon and (hopefully) applied to the source, I will send out a regression test. ~Mike ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

[PATCH] Adds ly:engraver-property to engraver-scheme.cc

Sorry for the slew of patches in this vein - as I run into problems while trying to create a real engraver in scheme, I'm sending these patches along. http://codereview.appspot.com/1878048 Cheers, ~Mike 0002-Adds-ly-engraver-property-to-engraver-scheme.patch Description: Binary

[PATCH] ly:define-event-class proposal

Hey all, I wrote the attached bit of code to programmatically create event classes. It does the job for my purposes, but I'm not sure if it is the best way to do this. In any case, I'd like to see this functionality become part of lilypond if possible! Cheers, ~Mike P.S. On R

Extra check in define-event-class patch

x27;s guide if people feel that'd be helpful - I think it would given a recent thread on frogs concerning engravers & their complexity), but I don't know if I should submit it as part of this patch set or as a new issue on Reitveld. Lemme know :-) ~Mike 0002-Symbol-checking.patch

Re: Extra check in define-event-class patch

On 8/12/10 7:20 PM, "Trevor Daniels" wrote: > > Mike Solomon wrote Thursday, August 12, 2010 10:16 AM > >>I added an extra symbol check to my event class patch: >> >> http://codereview.appspot.com/1867050 >> >> Also, I have a regression

Re: woodwind diagrams in lilypond (http://codereview.appspot.com/1385041)

ainly do guided and specific work that would contribute towards people's understand of these diagrams. Specifically, if there's one person who'd like to work on the dox for this, I can be in touch with him or her directly about how this feature works. ~Mike On 8/9/10 6:57 PM, &quo

Re: Change syntax for woodwind-diagram markup (issue1946043)

Hey all, Looks a-ok to me: I don't have time to run the regtests now (still traveling, no battery...), but if they don't break, feel free to push! ~Mike On 8/15/10 12:10 AM, "carl.d.soren...@gmail.com" wrote: > OK, I fixed all these things, but I did have a questi

Re: Extra check in define-event-class patch

Should be good to go! ~Mike (issue 1867050) On 8/14/10 10:20 AM, "Neil Puttock" wrote: > On 13 August 2010 23:31, Neil Puttock wrote: > >> I'll see if it works tomorrow, otherwise I'll copy them here. > > Still no joy. It looks like the base file

Re: Extra check in define-event-class patch

es about registering syntax changes so that users can update their files accordingly. http://codereview.appspot.com/1867050 Thanks! ~Mike On 8/18/10 6:22 PM, "Neil Puttock" wrote: > On 18 August 2010 23:12, Mike Solomon wrote: >> Should be good to go! >> ~Mike >

Re: Dynamics context causes "couldn't fit music on page"

This makes me wonder...if one creates a grob from the scheme-end, is there a way that it can get in on the pure-print-callback action? Currently, the list is not defined publicly. A simple define-public should do the trick, but I’m not sure...thoughts? ~Mike On 8/19/10 3:28 PM, "Joe N

Re: Dynamics context causes "couldn't fit music on page"

Done. http://codereview.appspot.com/1983047 On 8/19/10 7:52 PM, "Joe Neeman" wrote: > 2010/8/19 Mike Solomon >> This makes me wonder...if one creates a grob from the scheme-end, is there a >> way that it can get in on the pure-print-callback action?  Currently, t

Re: Make pure-print-callbacks public (issue1983047)

sts that should have getter/setters and I'll write the patches accordingly. Cheers, Mike On 8/20/10 12:00 PM, "joenee...@gmail.com" wrote: > While this patch looks fine to me, there are a bunch of other lists with > related functionality. You may want to make some of them

Re: Make pure-print-callbacks public (issue1983047)

Adder function created. http://codereview.appspot.com/2020041 On 8/20/10 12:29 PM, "Mike Solomon" wrote: > I don't know the source well enough to know the category of lists to which > you're referring. However, I don't mind making an adder function for this >

Re: woodwind diagram docs

ring?"), but if you sent a list of pertinent properties it'd make it more like that I get it right the first time. If someone else would care to work on this before me (or if someone would like to get to know the code by working on this with me), don't hesitate to chime in. Ch

programmatic pure-print stumbling block

ation)? (b) to register the function in various places (what parts of lilypond aside from the list pure-print-callbacks in scm/define-grobs.scm need to know about these functions)? Thanks! ~Mike ___ lilypond-devel mailing list lilypond-devel@gnu.org

Odd messaage from 2.13.32

Hey all, Just got 2.13.32 running and got the following message: warning: couldn't fit music on page: overflow is -0.00 I think it's good to get a warning message when there's spillover, but it seems odd that an overflow of 0 would solicit a warning message. T

Re: Pure print callback adder function. (issue2020041)

s that pure print cannot touch. None of this is necessarily an impediment to moving forward if you feel that the user should just be careful, but it does run the risk of leaving an unchecked way in lilypond to cause a huge time drain. ~Mike On 9/6/10 6:35 AM, "carl.d.soren...@gmail.com" wro

Woodwind update

imate). Please hit me back if you have some time on your hands and would like to help me improve this feature. Cheers, Mike ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: tunefl and other web services

nd be able to work on them and show them remotely. Cheers - Mike PS - the girl? a friend - I got the idea after seeing this Debian linux ad: http://2.bp.blogspot.com/_29YQGGYHw90/TFrnCsmnI_I/AXY/Kc11T499a-g/s1600/debian.jpg Nothing wrong in using beauty to promote beautiful engraving,

cc1plus warning

I'm getting the following warning compiling on my new mac : cc1plus: warning: command line option "-Wstrict-prototypes" is valid for Ada/C/ObjC but not for C++ Any ideas where that's coming from? Cheers, MS ___ lilypond-devel mailing list lilypond-dev

Re: Woodwind diagram for recorder

On 29 mai 2013, at 17:30, Dominique Faure wrote: > Hi, > > Few months ago, I suggested Mike Solomon (which was asking for work > during flight time) to add the regular recorder in the woodwind > diagram module, but for some reasons the request remained unanswered. > > H

Re: Path to LilyPond 2.18

On 6 juin 2013, at 06:52, Keith OHara wrote: > Phil Holmes philholmes.net> writes: > >> I believe we need to get fixes for 3363 and 3386 into the code base before >> cutting a release candidate. > > Issue 3363, the scripts on cross-staff things being grossly misplaced, > looks simple, but

Re: REQUEST: \whiteout using grob stencil instead of bounding box

; or whatever) property, and have it simply "outline" the grob. Of course, if > this were implemented, it would be great to retain the current behaviour > (i.e., the bounding box whiteout). > > Thanks, > Kieren. > > p.s. Mike: If you've got time for some more

Re: The Freeze

On 1 juil. 2013, at 11:42, Janek Warchoł wrote: > Hi all, > > it's been a while since we announced the freeze aimed at releasing > 2.18. I'm not sure if it's working properly, though: it seems that > the development process lost a lot of momentum. (As for the remaning > critical bugs, i feel

Working with unpure-pure containers (issue 3385)

Hey all, Sorry for my lack of devel time over the past few months - I'm using LilyPond almost every day but just don't have time to develop. Once I get some, I'm very much looking forward to getting back into things. I've created a remote branch dev/upc-test that rewinds master to the commit

Re: what does LedgerLineSpanner.thickness do?

On 13 août 2013, at 08:38, Mark Polesky wrote: > Ledger line thickness is set with the 'ledger-line-thickness > property of the StaffSymbol grob. So then why does the > LedgerLineSpanner grob have a 'thickness property? > > - Mark Probably cruft - I don't see it being looked up anywhere in t

Re: untangling the remaining regressions

On 13 août 2013, at 11:33, "Keith OHara" wrote: > When solving regressions we have the choice to go forward or to retreat -- > to repair, or revert, the change that caused the regression. > > (1) The positioning of fingerings using the outline of the note-column (the > last patch under issue 25

Re: untangling the remaining regressions

On 13 août 2013, at 11:59, David Kastrup wrote: > Mike Solomon writes: > >> The short term solution may be to create an internal property called >> "pure-relevant", set it to #t as default for all grobs in the >> Grob::Grob constructor and set to fals

Re: untangling the remaining regressions

On 13 août 2013, at 12:41, David Kastrup wrote: > We don't need a > whole reimplementation of old code for the sole purpose of temporarily > masking bugs with the new implementation by retro-fitting a distant > clone of the old implementation on top of the unfixed new > implementation. I put up

Re: untangling the remaining regressions

On 14 août 2013, at 09:58, Keith OHara wrote: > > > Mike posted three patches to issue 3385 that attempted to go halfway > backwards, partially restoring the pure-relevant? test, but these > ran into regressions. > > Going forward would be to make the pure versio

Re: Avoids calling pure calculations too far downstream and avoids triggering quanting too far upstream (issue 12951044)

On 15 août 2013, at 08:35, k-ohara5...@oco.net wrote: > Of course this is two commits. > probably three (axis-group-interface.cc and two independent changes in side-position-interface.cc). > I do not know of any reason to distinguish the concept of 'cross-staff' > from your new > height_depend

Re: Avoids calling pure calculations too far downstream and avoids triggering quanting too far upstream (issue 12951044)

On 15 août 2013, at 08:51, k-ohara5...@oco.net wrote: > On 2013/08/15 05:35:49, Keith wrote: > >> In most cases, 'cross-staff' means exactly that the height (Y-extent) > depends on >> staff-spacing. > > Oh. It is the ottava bracket from the example for issue 3385 that is > not cross-staff, > b

Re: Avoids calling pure calculations too far downstream and avoids triggering quanting too far upstream (issue 12951044)

On 15 août 2013, at 09:03, k-ohara5...@oco.net wrote: > On 2013/08/15 05:51:36, mike7 wrote: >> On 15 août 2013, at 08:35, mailto:k-ohara5...@oco.net wrote: > >> > Could you mark the relevant object as 'cross-staff' when the > cross-staff >> > item is added to it 'support', rather than search th

Re: Tests for either cross-staff or supported by cross-staff stems to skip elements in pure side-positi… (issue 12921043)

On 15 août 2013, at 10:29, k-ohara5...@oco.net wrote: > > https://codereview.appspot.com/12921043/diff/3001/lily/side-position-interface.cc > File lily/side-position-interface.cc (right): > > https://codereview.appspot.com/12921043/diff/3001/lily/side-position-interface.cc#newcode278 > lily/sid

Re: untangling the remaining regressions

ure' function calls) but they are most often ignored. This is the basis of all my stub work (SlurStub, etc.). > > All non-cross-staff grobs must find their positions while the positions of > cross-staff grobs remain hypothetical. Yup. > That is, any positions callbacks for a no

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 24 août 2013, at 10:18, lemzw...@googlemail.com wrote: > LGTM, except of using `surrogate' in the name. Given that the stencil > dimensions are actively overridden on request, I would like rather like > having `override'. For me, the word `surrogate' suggests something > passive. > > https:

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 24 août 2013, at 13:38, d...@gnu.org wrote: > On 2013/08/24 10:31:38, dak wrote: >> On 2013/08/24 10:05:17, mike7 wrote: >> > >> > The stencil command takes the skyline of stencil X and uses it as a >> replacement >> > for skyline Y. We could use replacement-skyline-stencil as the > command. >

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 24 août 2013, at 16:36, d...@gnu.org wrote: > On 2013/08/24 13:19:30, mike7 wrote: > >> The question I'm asking is "How can I allow the user to replace the > skyline of a >> stencil with another shape?" > > I got that. But what is the actual use case for that? Before skylines > became a th

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 25 août 2013, at 09:15, k-ohara5...@oco.net wrote: > On 2013/08/24 16:33:39, MikeSol wrote: > >> On 2013/08/24 16:19:27, dak wrote: > >> > Stupid question: _iff_ you store a _whole_ replacement skyline > anyway, >> > shouldn't pad-around just shift the original left/right/up/down > skylines >

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 25 août 2013, at 16:04, d...@gnu.org wrote: > On 2013/08/25 08:22:01, mike7 wrote: >> On 25 août 2013, at 09:15, mailto:k-ohara5...@oco.net wrote: > >> > A second stencil is not a very good data structure for reserving > space. >> > Skylines are better at this, and we can \once\override 'paddi

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 26 août 2013, at 07:36, d...@gnu.org wrote: > On 2013/08/26 04:08:17, mike7 wrote: >> On 25 août 2013, at 16:04, mailto:d...@gnu.org wrote: > >> > On 2013/08/25 08:22:01, mike7 wrote: >> >> On 25 août 2013, at 09:15, mailto:k-ohara5...@oco.net wrote: >> > >> >> > A second stencil is not a ver

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 26 août 2013, at 08:20, d...@gnu.org wrote: > On 2013/08/26 04:51:50, mike7 wrote: >> On 26 août 2013, at 07:36, mailto:d...@gnu.org wrote: > >> > On 2013/08/26 04:08:17, mike7 wrote: >> >> On 25 août 2013, at 16:04, mailto:d...@gnu.org wrote: >> > >> >> > On 2013/08/25 08:22:01, mike7 wrote:

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 26 août 2013, at 08:35, d...@gnu.org wrote: > On 2013/08/26 05:25:42, mike7 wrote: >> On 26 août 2013, at 08:20, mailto:d...@gnu.org wrote: > >> > Indeed. Instead of calling stencil-integrate on a "surrogate > stencil" >> > or whatever for deriving a skyline, the respective stencil operation

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 26 août 2013, at 11:41, d...@gnu.org wrote: > On 2013/08/26 06:42:38, mike7 wrote: >> On 26 août 2013, at 08:35, mailto:d...@gnu.org wrote: > >> > So why do you talk about that in the first place? > >> Because I'd rather spend more time implementing a solid system than > less time >> impleme

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 26 août 2013, at 12:39, Werner LEMBERG wrote: > >> To summarize: stencil expressions (expr_) currently do not contain >> information for how to place stencils with respect to others. They >> contain information passed to the backends on how to lay out >> stencils. If we adopt either Keith'

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 26 août 2013, at 13:00, d...@gnu.org wrote: > That's bullshit since nobody assembles stencil expressions manually. I assemble stencil expressions manually. > This is done using opaque functions like ly:stencil-translate. ly:make-stencil is all over the place in the code and is a public func

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 26 août 2013, at 13:13, d...@gnu.org wrote: > On 2013/08/26 10:05:25, mike7 wrote: >> On 26 août 2013, at 13:00, mailto:d...@gnu.org wrote: > >> > That's bullshit since nobody assembles stencil expressions manually. > >> I assemble stencil expressions manually. > > If you meddle with interna

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 26 août 2013, at 13:31, d...@gnu.org wrote: > On 2013/08/26 10:18:04, mike7 wrote: > >> I think Harm is right that, irrespective of documentation, users use > what's >> available. > > If people meddle with internals instead of using provided API functions, > they deserve whatever they get. >

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 27 août 2013, at 07:18, "Keith OHara" wrote: > On Mon, 26 Aug 2013 02:11:28 -0700, Mike Solomon wrote: > >> >> I think that if we can devise a better system, the creation of a new stencil >> primitive is unnecessary. If we agree that we should impleme

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 27 août 2013, at 09:01, "Keith OHara" wrote: > On Mon, 26 Aug 2013 22:37:17 -0700, Mike Solomon wrote: > >> On 27 août 2013, at 07:18, "Keith OHara" wrote: >> >>> (1) At the moment, the stencil expressions are assembled first, when markup

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 27 août 2013, at 11:09, d...@gnu.org wrote: > On 2013/08/27 06:58:05, mike7 wrote: >> On 27 août 2013, at 09:01, "Keith OHara" > wrote: > >> > How are skylines asked-for, and is it possible to know if they were > asked for >> while interpreting the markup ? >> > {

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 28 août 2013, at 05:28, "Keith OHara" wrote: > On Mon, 26 Aug 2013 23:57:58 -0700, Mike Solomon wrote: > >> On 27 août 2013, at 09:01, "Keith OHara" wrote: >> >>> On Mon, 26 Aug 2013 22:37:17 -0700, Mike Solomon >>> wrote: >>

Re: Adds dimension stencil command to correct with-dimension (issue 12957047)

On 28 août 2013, at 09:47, "Keith OHara" wrote: > On Tue, 27 Aug 2013 23:30:32 -0700, Mike Solomon wrote: > >> On 28 août 2013, at 05:28, "Keith OHara" wrote: >> >>> Of course I think it would be better to allow box dimensions in the stenc

Re: Allows inner-markup spacing using skylines. (issue 13341044)

ever cached, but as it > is not >> default beahvior anywhere in the code base, it allows people to decide > on the >> efficiency versus snugness-of-placement trade-off. For people > creating complex >> markups, this is useful. > > Mike, the question is not whether

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

On 30 août 2013, at 07:49, "Keith OHara" wrote: > On Thu, 29 Aug 2013 21:24:25 -0700, wrote: > >> Conceptually, I prefer (1), but this is based on your descriptions and >> the previous discussion, not on understanding the code... >> > > Then look at (2) and see if you think that would be goo

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

On 30 août 2013, at 10:11, d...@gnu.org wrote: > On 2013/08/30 07:41:49, mike7 wrote: > >> I'd still argue that (2) is the best way to go as it is a > one-stop-shopping way >> to clear all these bugs (and perhaps more) as they arise. > > Since we are currently in the process of recovering from t

Re: stencil-integral: use box extents specified in markup; issue 3255 (issue 9295044)

On 30 août 2013, at 11:39, d...@gnu.org wrote: > >> > I suggest that we refrain from embracing your somewhat optimistic >> > estimates until after wrapping up a stable release. > >> I don't see the relationship between the two. > > Which makes it a good thing that you are not pursuing a career

Re: Checks to see if tuplet brackets have bounds (issue 13582046)

On 13 sept. 2013, at 08:41, d...@gnu.org wrote: > > https://codereview.appspot.com/13582046/diff/1/lily/tuplet-bracket.cc > File lily/tuplet-bracket.cc (right): > > https://codereview.appspot.com/13582046/diff/1/lily/tuplet-bracket.cc#newcode99 > lily/tuplet-bracket.cc:99: if (!left || !right)

Re: Checks to see if tuplet brackets have bounds (issue 13582046)

s one already exists. At least that, I >> believe, was the general approach adopted in the past and it seems >> sound to me. > > Well, being defensive is always a good idea. It's just that when we get > to _know_ the defenses to be triggered without us expecting it, we

patch: doc typo fixes/etc

smooth out the comparison to other languages. Technically, it's not the case, for example, that most programming languages use { } to enclose expressions. Regards, Mike From c367521fe65ff61c50dcb918ce35cf31613ccebf Mon Sep 17 00:00:00 2001 From: Michael Coleman <[EMAIL PROTECTED]> Date: Wed,

Plug Ins...

Hello, my name is Mike I sent a message a few days ago concerning plug ins for lilypond and have received no response. Can anyone tell me who might be able to help with this? I would like the stave lines to be built from consecutive rows of letters and to change the colour of each line. (line 1

plug ins...

BBB GG EE In chapter 3 'basic features' I found reference to st to suppress property, but I can not get into the folder to modify the lines to letters Can anyone help with this? Cheers, Mike ___

Re: release plans

Sorry for the slow response - I'm having trouble with my copy of the source (too much tinkering...not enough sleep...). Could someone please run the appropriate regtests on Neil's patch and see if it breaks anything? Cheers, MS On 9/22/10 12:11 PM, "Graham Percival" wrote: > Here's an update

flex issue when compiling

n this problem? If so, is there a flag I could pass to flex that'll clear this up? Should I be sending this to the flex list? Thanks! Mike ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Fix 1456 (issue3693042)

Tested and nothing broke. Let me know if you want me to send any output. Cheers, M "I withhold the right to brag about how fast my new computer is until it becomes obsolete in 2 weeks" S On Dec 17, 2010, at 11:57 AM, "Phil Holmes" wrote: > - Original Message - From: > To: > Cc: ;

Re: Lily dances

On Dec 22, 2010, at 5:10 PM, Federico Bruni wrote: > 2010/12/22 Mike Solomon > Hey all, > > I am embarking on a new project to make lily dance. > > Attached are 3 files: > > 1) svgdance.svg (best viewed in something that's not Internet Explorer - > click on

Re: Lily dances

On Dec 22, 2010, at 10:21 PM, Jan Warchoł wrote: > 2010/12/22 Mike Solomon > 1) svgdance.svg (best viewed in something that's not Internet Explorer - > click on the notes and/or accidentals and see what happens!) > > Cool! Thanks for sharing :) > > cheers, >

Re: a plea to new contributors

Graham, Looks like Joe fixed this (please correct me if I'm wrong). I'll keep my eyes out for new critical issues before I take off. Cheers, MS On Jan 2, 2011, at 2:21 AM, Graham Percival wrote: > On Sat, Jan 01, 2011 at 12:21:24PM +0100, m...@apollinemike.com wrote: >> >> I have a flight on J

Potential issue 39 fix w/ patch

I've included before and after photos.Cheers,MS 0001-Potential-fix-for-issue-39.patch Description: Binary data <><>___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Score follower

to a file using Scheme. If this sounds like Chinese, lemme know and I can try to cook up a simple proof of concept depending on what you want to do. Cheers, Mike On Jan 5, 2011, at 11:40, Carl Sorensen wrote: > > > > On 1/5/11 8:44 AM, "Konstantin" wrote: >

Re: Potential issue 39 fix w/ patch

arises when it causes ambiguity in the rhythm. Anyway, I'll get on your suggestion and will leave it to others to decide which one is prettier. In neither case would the solution be very computationally expensive. Cheers, Mike On Jan 5, 2011, at 23:49, Keith OHara wrote: > Mike S

Re: Potential fix for issue 37

Kinda meh, but it gets the job done!  I've included three patches, including the original.The second preserves the flat beams but gets rid of the crashing lily, whereas the third implements some bendiness.Cheers,MS 0003-Second-pass-on-potential-fix-for-issue-37.patch Description: Binary data 000

Re: Fixing issue 37 with extra position callback (issue3928041)

en...@gmail.com wrote: > Reviewers: MikeSol, > > Message: > Looks great, Mike! > > You have some code indentation issues that don't match our style. Other > than that, I think it's good to go. > > Of course, we do need a regression test as well. > >

Re: Fixing issue 37 with extra position callback (issue3928041)

All is fixed. Attached is the original patch plus one with Carl's suggested formatting changes. I ran all of the regtests and nothing breaks. I'd like one other person to test these patches out w/ the attached .ly document to confirm. Then, if there are no other other comments, please push t

Build failure with make check

deps-box-padding=3.00 -dread-file-list -dno-strip-output-dir "/Users/mikesolomon/devel/lilypond/out/lybook-testdb/snippet-names--5659578003698203985.ly" Child was terminated by signal 11 make[2]: *** [out-test/collated-files.texi] Error 1 make[1]: *** [local-test] Error 2 make: *** [test] Err

Re: Fixing issue 37 with extra position callback (issue3928041)

Hey Keith, I believe the new patch set fixes that problem. Cheers, MS On Jan 8, 2011, at 9:01 PM, Keith OHara wrote: > Mike, > I like it on small scores. > > I let your patch compile a symphony and got several unneeded stem > lengthenings per page. I started carving out a r

Re: Fixing issue 37 with extra position callback (issue3928041)

-collision-engraver.cc#newcode4 > lily/beam-collision-engraver.cc:4: Copyright (C) 1997--2010 Han-Wen > Nienhuys > Copyright 2011 Mike Solomon > > http://codereview.appspot.com/3928041/diff/17001/lily/beam.cc > File lily/beam.cc (right): > > http://codereview.appspot.com/39

Re: Potential issue 39 fix w/ patch

.Cheers,MS 0002-Fixes-issue-39-by-raising-stems.patch Description: Binary data 0001-Potential-fix-for-issue-39.patch Description: Binary data flag-collision.pdf Description: Adobe PDF document On Jan 7, 2011, at 5:42 PM, Keith OHara wrote: Original Message ----From: "Mike

Erroneous patch set for the previous issue 37 email - correct one here

Cheers, MS 0006-Better-reflects-what-is-actually-under-the-beam.patch Description: Binary data 0005-Attempt-to-deal-with-smaller-font-sizes-like-cue-not.patch Description: Binary data 0004-Changes-suggested-by-Carl.patch Description: Binary data 0003-Intermediary-37-patch.patch Description

Re: CueVoice issue fixed

Ok. The problem still remains, though, for the one-measure-long piece in \time 2000/4 . Although absurd, it is possible. Cheers, MS On Jan 12, 2011, at 10:28 AM, Carl Sorensen wrote: > > > > On 1/11/11 5:30 PM, "Mike Solomon" wrote: > >> Unfortunately, I

Ran the regtests for issue 39

The only change I can see is in collision heads, which adds breathing room between the stem and the flag (I've attached old and new). I am not an expert on how much room should be in there - I leave it to someone else to fiddle with the magic numbers in note-collision.cc to get it right. I opt

Re: Fixes issue 39 by raising stems (issue3934041)

Thanks Keith, I think that the case you're talking about (<< e'32. \\ e'2>>) is a problem with my code, which triggers stem raising for flags that fall on the right even though these flags do note cause intersection problems. Is there a good way to weed this out? On Jan 13, 2011, at 1:03 AM,

Re: Potential fix for issue 37

dding that please? > > Jan > > Mike Solomon schreef op vr 07-01-2011 om 20:54 [-0500]: >> Kinda meh, but it gets the job done! I've included three patches, >> including the original. >> The second preserves the flat beams but gets rid of the crashing

Re: biweekly Critical issues plea

acker (2.13.46). Graham identifies that the output was correct on the bugtracker (2.12.3). 1/19 Mike confirms that the regression is indeed due to 1190 and realizes that he is not subscribed to the bug list. Mike proposes a patch based on the discussion between Neil and Keith, which is attached to t

Re: biweekly Critical issues plea

; > On 1/19/11 4:33 PM, "Mike Solomon" wrote: > >> Got it. >> >> Then, here is the state of things: >> >> 1/6 >> Bug is first reported on the bug list. >> >> 1/7 >> Neil reports adding a default 'extra-spacing-height t

Re: Fixing issue 37 with extra position callback (issue3928041)

x27;d lead to a very squashed result. I'm just not sure how low is too low before you have to give up and print a collision. Cheers, MS On Jan 24, 2011, at 7:46 AM, Han-Wen Nienhuys wrote: > On Mon, Jan 24, 2011 at 9:42 AM, Mike Solomon wrote: >> Fixed, although I have no idea what

Issue 37 - new work

t how to do that in that particular scenario. Otherwise, all of Han-Wen's concerns are addressed. And, as before, the truly heinous collisions are unavoidable because there is no way to anticipate desired output. I still need to do a make check. Cheers, Mike 0002-Triggers-second-q

New patch set w/ expanded regtest

Hey all, 0003-Tweaks-regtest-and-adds-avoid-collisions-property.patch Description: Binary data 0002-Triggers-second-quant-pass-for-collisions.patch Description: Binary data 0001-Implements-a-more-robust-solution-to-issue-37.patch Description: Binary data I still can't post properly to Riet

Multimeasure rest print function

Hey all, I noticed the following lines of code in Multi_measure_rest::print (SCM smob): int measures = 0; SCM m (me->get_property ("measure-count")); if (scm_is_number (m)) measures = scm_to_int (m); It seems that the variable `measures' is unused in this function - can this be de

My issue 37 work finally on Rietveld after fixing mimetypes issue

http://codereview.appspot.com/4022045 Cheers, MS ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Issue 37 - new work

On Jan 28, 2011, at 7:45 AM, Han-Wen Nienhuys wrote: > On Thu, Jan 27, 2011 at 12:47 AM, Mike Solomon wrote: >> Hey all, >> >> I have a new Issue 37 fix. >> >> The attached patch set implements a 2-pass approach through the quanting >> that is potentiall

Re: Issue 37 - new work

<> I cooked up this musical example that shows both responses to upward and downward pressure to give you an idea of where I'm coming from. Is there a way to get this type of collision avoidance w/o a 2nd quanting pass? Cheers, ~Mike On Jan 28, 2011, at 7:45 AM, Han-Wen Nienhuys w

Re: Issue 37 - new work

On Jan 28, 2011, at 6:12 PM, Mike Solomon wrote: > On Jan 28, 2011, at 1:15 PM, Han-Wen Nienhuys wrote: > >> pressed send too soon. >> >> On Fri, Jan 28, 2011 at 4:14 PM, Han-Wen Nienhuys wrote: >>> On Fri, Jan 28, 2011 at 11:22 AM, Mike Solomon wrote: &g

Re: PATCHES: 48-hour notice for note spacing and negative frets

On Feb 2, 2011, at 9:32 PM, Han-Wen Nienhuys wrote:On Thu, Feb 3, 2011 at 12:26 AM, m...@apollinemike.com wrote:The most recent patch set only has a single pass through beam quanting.  Idon't believe it adds significant overhead to a score's compile time,although I'd need som

Re: PATCHES: 48-hour notice for note spacing and negative frets

scans are posted. How should I go about sharing it w/ all interested parties & staying w/in the realm of what's legal @ the same time? Cheers, MS On Feb 3, 2011, at 9:41 AM, Han-Wen Nienhuys wrote: > > > On Thu, Feb 3, 2011 at 11:58 AM, Mike Solomon wrote: > > In

Re: PATCHES: 48-hour notice for note spacing and negative frets

at 10:55 AM, David Kastrup wrote: > Mike Solomon writes: > >> Hey all, >> >> I have scanned examples from Boulez's Third Piano Sonata, Elliot >> Carter's Night Fantasies, Elliot Carter's Sonata, and Stockhausen's >> Klavierstücke II & I

Re: Introduce Beam_scoring_problem::quant_range, for quickly filtering out invalid positions. (issue4129050)

On Feb 4, 2011, at 10:19 AM, Han-Wen Nienhuys wrote: > On Fri, Feb 4, 2011 at 10:08 AM, wrote: >> LGTM, but I can't do a regtest today :( >> >> >> http://codereview.appspot.com/4129050/diff/1/lily/beam-quanting.cc >> File lily/beam-quanting.cc (right): >> >> http://codereview.appspot.com/4129

Re: Introduce Beam_scoring_problem::quant_range, for quickly filtering out invalid positions. (issue4129050)

On Feb 5, 2011, at 7:40 AM, Han-Wen Nienhuys wrote: > On Fri, Feb 4, 2011 at 1:30 PM, Mike Solomon wrote: >> On Feb 4, 2011, at 10:19 AM, Han-Wen Nienhuys wrote: >> >>> On Fri, Feb 4, 2011 at 10:08 AM, wrote: >>>> LGTM, but I can't do

Feature? Bug? (part 1)

Hey all, In the attached png, you'll see that the feathered beam resets to 0 feather @ the line break. Actually, it even regresses a bit: you'll see that it's actually fatter at the linebreak than it is a couple beats after, after which it starts to spread out again. I would argue that the la

Feature? Bug? (part 2)

Well, no, I lied, just plain bug. I can't imagine this being a feature. But, before I start working on it, does anyone have a clean work-around to get the bendAfter spanning the line break correctly? Cheers, MS <> \score { \new Staff << { s1*3 \break } \relative c' { s1*2 \times 44/72 {\cresc

<    1   2   3   4   >