Re: LSR is now on 2.14

2012-04-01 Thread David Nalesnik
Hi, Simplest way might be for Harm, David and myself to put these in the LSR. Don't think it'll take that long. I'd then check and approve them - so if I took 10 and you 2 share the others out, that should be fair? What do you think? Sure, I'll be happy to take a share. -David

Re: LSR is now on 2.14

2012-04-01 Thread David Nalesnik
Hi Harm, On Sun, Apr 1, 2012 at 3:23 PM, Phil Holmes m...@philholmes.net wrote: From: Thomas Morley thomasmorle...@googlemail.com** To: Phil Holmes m...@philholmes.net Cc: David Nalesnik david.nales...@gmail.com; lilypond-devel lilypond-devel@gnu.org Sent: Sunday, April 01, 2012 6:19 PM

Re: LSR is now on 2.14

2012-04-02 Thread David Nalesnik
Hi, On Mon, Apr 2, 2012 at 11:48 AM, Phil Holmes m...@philholmes.net wrote: Sorry about the top post. Windows mail is sometimes a pain. I'll do fretted-string-harmonics... to screech-boink. Phil Holmes Attached the tarball with the missing files from `Ducumentatione/snippets/new/'

Re: LSR is now on 2.14

2012-04-03 Thread David Nalesnik
Hi Harm, On Tue, Apr 3, 2012 at 8:45 AM, Thomas Morley thomasmorle...@googlemail.com wrote: Hi David, I just checked our recent addings to the LSR. In unfretted-headword.ly I noticed the line: noflag = \once \override Flag #'style = #'no-flag could you change it to: noflag

Re: Center a number above a measure - Issue 2445

2012-04-12 Thread David Nalesnik
Hi James, On Wed, Apr 11, 2012 at 4:28 AM, James pkx1...@gmail.com wrote: David, I saw that Colin had opened a tracker for your *ly file http://code.google.com/p/lilypond/issues/detail?id=2445 I'm not a code developer, however I think it might be better to also open a Rietveld issue so

Re: measure counter engraver

2012-04-24 Thread David Nalesnik
David, On Tue, Apr 24, 2012 at 12:41 AM, David Kastrup d...@gnu.org wrote: David Nalesnik david.nales...@gmail.com writes: The examples are written to work with the latest versions. (I'm using the current release candidate.) Then one could use make-engraver. True, true. My motivation

Re: measure counter engraver

2012-04-24 Thread David Nalesnik
David, .Any problems which might result will only affect that multi-file run? Basically every use of lilypond-book is a multi-file run. So are our regtests. Understood. I also won't upload this to the LSR at the current time since it could potentially cause annoyances the next time it

Re: measure counter engraver

2012-04-24 Thread David Nalesnik
Hi Phil, On Tue, Apr 24, 2012 at 9:13 AM, Phil Holmes m...@philholmes.net wrote: ** You're OK uploading it to the LSR providing you don't tag it with docs - it's only those that are imported into the documentation system. I'm just speaking here from the experience of checking a bunch of LSR

Re: measure counter engraver

2012-04-25 Thread David Nalesnik
Hi, Unrelated issue: I notice that the original post with attached files hasn't shown up on archive 2 on either -user or -devel, even though I sent that post 13 hours ago. It appears on archive 1, but there the .ly file appears as a binary data file, which I cannot open. Am I guilty of

Re: Get the positions of a beam (or other grob with positions)

2012-05-19 Thread David Nalesnik
Hi Janek, On Sat, May 19, 2012 at 8:03 AM, Janek Warchoł janek.lilyp...@gmail.comwrote: Hi David all, i want to make arpeggios longer (so that they overshoot the chords a bit). I've modified your offsetBeamPositions function to work on Arpeggios instead of Beams (that was easy), but i'm

Re: use guile to set specific extra-offset values if stem direction is up or down

2012-05-21 Thread David Nalesnik
Hi José, On Mon, May 21, 2012 at 9:09 AM, padovani zepadovani.li...@gmail.comwrote: Hello, I'm dealing with special noteheads I've designed and I need to move the stem and the flag differently if stem direction is up or down. I would like to create a guile lisp clause that sets the

Re: use guile to set specific extra-offset values if stem direction is up or down

2012-05-21 Thread David Nalesnik
Hi José, On Mon, May 21, 2012 at 10:20 AM, padovani zepadovani.li...@gmail.comwrote: Em 5/21/12 11:44 AM, David Nalesnik escreveu: \override Stem #'extra-offset = #(lambda (grob) (if (eqv? (ly:stem::calc-direction grob) UP) '(-0.02 . -0.25) '(0.02 . 0.25))) Hello David, thank

Re: use guile to set specific extra-offset values if stem direction is up or down

2012-05-21 Thread David Nalesnik
José, On Mon, May 21, 2012 at 11:02 AM, padovani zepadovani.li...@gmail.comwrote: Oh, I'm sorry... I was reading everything in a hurry and didn't got the second part of your message. No problem! I learned something new about ly:stem::calc-direction because of your email: didn't know that it

Re: use guile to set specific extra-offset values if stem direction is up or down

2012-05-21 Thread David Nalesnik
'' { %\override Stem #'color = #red \stemUp c8 \noBeam \customStemAndFlag c \stemDown c \noBeam \customStemAndFlag c } HTH, David Thank you again, José Em 5/21/12 1:10 PM, David Nalesnik escreveu: José, On Mon, May 21, 2012 at 11:02 AM, padovani zepadovani.li...@gmail.com

Re: use guile to set specific extra-offset values if stem direction is up or down

2012-05-21 Thread David Nalesnik
Hi again, It would be possible to offset both of them, but I think you might be better served by approaching the problem from a different angle. You can control the relative positioning of stem and note head through 'stem-attachment, which is a property of NoteHead. The flag will move

Re: Developing for LilyPond

2012-05-23 Thread David Nalesnik
Hi Mike, On Wed, May 23, 2012 at 12:47 AM, m...@mikesolomon.org m...@mikesolomon.org wrote: [ ... ] Your patch is a post-processing corrective for when this fails. However, the goal of LilyPond is twofold: 1) Provide users with the ability to tweak LilyPond's output when for some reason

add \shape (issue 6255056)

2012-05-28 Thread david . nalesnik
Reviewers: , Message: Add \shape as a method of modifying the control points of bezier curves. Description: add \shape add \shape as a method of altering bezier curves by offsetting their control-points. Please review this at http://codereview.appspot.com/6255056/ Affected files: A

Re: add \shape (issue 6255056)

2012-05-28 Thread david . nalesnik
On 2012/05/28 20:23:37, janek wrote: generally LGTM. What about coloring the \shaped curves? This would make checking regtests more obvious to an ordinary person who may be not familiar with this stuff. Good idea! I did this for both regression tests.

Re: add \shape (issue 6255056)

2012-05-31 Thread david . nalesnik
Thank you for reviewing this, Neil! On 2012/05/31 16:22:53, Neil Puttock wrote: http://codereview.appspot.com/6255056/diff/5001/input/regression/shape-slurs.ly File input/regression/shape-slurs.ly (right):

Re: add \shape (issue 6255056)

2012-06-03 Thread david . nalesnik
On 2012/06/03 19:39:35, janek wrote: Hi David, please close this Rietveld issue (crossed circle next to its name). OK, done. Thanks! David http://codereview.appspot.com/6255056/ ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: generalizing offsets to properties

2012-06-07 Thread David Nalesnik
Hi Janek, On Tue, Jun 5, 2012 at 12:55 AM, Janek Warchoł janek.lilyp...@gmail.comwrote: David already knows my opinion ;) - i'd just like to share with you how this function made my day yesterday! I have a song with piano accompaniment which contains loads of portato passages, and they were

Re: get rid of minimum-[XY]-extents (issue 6294086)

2012-06-19 Thread david . nalesnik
, for an illustration of such modifying functions see this email from David Nalesnik: http://lists.gnu.org/archive/html/lilypond-devel/2012-06/msg00102.html http://codereview.appspot.com/6294086/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https

Re: shape and chords

2012-06-24 Thread David Nalesnik
Hi Jan-Peter, On Wed, Jun 20, 2012 at 2:46 AM, Jan-Peter Voigt jp.vo...@gmx.de wrote: Hi David, ... moving this to devel ... I had a look into tie-column.cc: In the scheme-callback calc_positioning_done all ties in a chord are assigned control-points in a loop. So it might be possible to

Re: shape and chords

2012-06-25 Thread David Nalesnik
Hi Jan-Peter, On Mon, Jun 25, 2012 at 4:47 AM, Jan-Peter Voigt jp.vo...@gmx.de wrote: Now the broken siblings are found, but only the first one is shaped. But it seems possible ... I will have another look after lunch ;-) I tried to fix what was amiss in your adaptation of \shape, but I

Re: shape and chords

2012-06-26 Thread David Nalesnik
Hi Jan-Peter, First of all, I agree with David Kastrup that this is a situation where a deep fix is in order so that simultaneous ties may be tweaked directly. Your example is working as expected :) Glad to hear it! To make it (more) usable, I think, we need wrapper functions, to avoid

Re: utf-8-strings

2012-07-08 Thread David Nalesnik
Hi Harm, I managed to drop about 300 lines, reducing it to a quarter of the original. Unfortunately, I'm getting Invalid UTF-8 string... warnings when I run your revised file. (I don't with the original...) -David ___ lilypond-devel mailing list

Re: utf-8-strings

2012-07-08 Thread David Nalesnik
On Sun, Jul 8, 2012 at 8:00 AM, David Kastrup d...@gnu.org wrote: Thomas Morley thomasmorle...@googlemail.com writes: 2012/7/8 David Nalesnik david.nales...@gmail.com: Hi Harm, I managed to drop about 300 lines, reducing it to a quarter of the original. Unfortunately, I'm

Re: utf-8-strings

2012-07-08 Thread David Nalesnik
On Sun, Jul 8, 2012 at 6:21 PM, Thomas Morley thomasmorle...@googlemail.com wrote: 2012/7/9 David Kastrup d...@gnu.org: David Nalesnik david.nales...@gmail.com writes: On Sun, Jul 8, 2012 at 8:00 AM, David Kastrup d...@gnu.org wrote: Thomas Morley thomasmorle...@googlemail.com

Re: utf-8-strings

2012-07-10 Thread David Nalesnik
Hi Harm, On Tue, Jul 10, 2012 at 3:07 PM, Thomas Morley thomasmorle...@googlemail.com wrote: (2) The code using the regexp library does not work with windows so far (perhaps never). Unfortunately, I'm still having no luck with Windows XP. (@David Kastrup: thank you for your suggestions and

Add function for overriding broken spanners to LilyPond. (issue 6397054)

2012-07-17 Thread david . nalesnik
Reviewers: , Description: Add function for overriding broken spanners to LilyPond. The music function \alterBroken is intended to facilitate overrides applied independently to the pieces of broken spanners--one of the perennial difficulties faced by users of LilyPond (addressed in Extending

Re: Add function for overriding broken spanners to LilyPond. (issue 6397054)

2012-07-18 Thread David Nalesnik
Hi Janek, On Wed, Jul 18, 2012 at 12:22 AM, Janek Warchoł janek.lilyp...@gmail.comwrote: On Wed, Jul 18, 2012 at 12:43 AM, david.nales...@gmail.com wrote: The function will override unbroken spanners, but it will ignore non-spanners with a warning. What about things like clef change at a

Re: Add function for overriding broken spanners to LilyPond. (issue 6397054)

2012-07-18 Thread David Nalesnik
On Wed, Jul 18, 2012 at 3:18 PM, David Kastrup d...@gnu.org wrote: Thomas Morley thomasmorle...@googlemail.com writes: 2012/7/18 Janek Warchoł janek.lilyp...@gmail.com: On Wed, Jul 18, 2012 at 8:18 PM, Thomas Morley thomasmorle...@googlemail.com wrote: 2012/7/18 Janek Warchoł

Re: Add function for overriding broken spanners to LilyPond. (issue 6397054)

2012-07-19 Thread David Nalesnik
Hi Janek, The only thing that worries me is the indentation - there are some tabs used and Frescobaldi displays them wrong (i suppose you meant 1 tab to be equal to 4 spaces, not 8). Oops! Usually I remember to convert tabs to spaces (and trim whitespace from the ends of lines), but not

Re: Add function for overriding broken spanners to LilyPond. (issue 6397054)

2012-07-19 Thread David Nalesnik
David, Why does this have to be a list? Why not just make it two separate arguments? If, for whatever reason, you really really really have this information in a two-element list called whatever, you can still write \once \alterBroken Staff.BarLine #'color #@whatever It certainly makes

Re: Add function for overriding broken spanners to LilyPond. (issue 6397054)

2012-07-20 Thread David Nalesnik
David, On Fri, Jul 20, 2012 at 12:38 AM, David Kastrup d...@gnu.org wrote: David Nalesnik david.nales...@gmail.com writes: David, Why does this have to be a list? Why not just make it two separate arguments? If, for whatever reason, you really really really have

Re: Add function for overriding broken spanners to LilyPond. (issue 6397054)

2012-07-20 Thread david . nalesnik
2012/7/18 Janek Warchoł janek.lilyp...@gmail.com: (however, it would be even more awesome if it was one general function for all purposes ;) ) Considering this some more, I think it's best at the moment to restrict the function to spanners. The expanded function encompassing spanners

Re: Add function for overriding broken spanners to LilyPond. (issue 6397054)

2012-07-21 Thread David Nalesnik
David, This use of \default for skipping optional arguments even in end position may appear a bit contrived. However, \key \default and \mark \default existed (hardwired in the parser) even before music functions ever heard of optional arguments, so I consider it an accomplishment to have

push patch for issue 2679

2012-07-24 Thread David Nalesnik
Hi, I don't have push ability, so would someone who does please push my patch for issue 2679 (Add function for overriding broken spanners to LilyPond) for me? Thanks so much, David ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
On Tue, Jul 24, 2012 at 8:56 AM, David Kastrup d...@gnu.org wrote: Phil Holmes m...@philholmes.net writes: I get the patch fails to apply - presumably because music-functions.scm has been updated twice since the patch was created? Does this mean it needs rebasing, or somesuch? Likely.

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
David, So, in this case, would I do something like: git pull -r then submit a new patch set? That's probably the easiest way. You'll likely get a merge conflict with instructions. Personally, I use Emacs and M-x smerge-ediff RET on the problematic file(s) for fixing the

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
David, Is there any reason I can't manually remove the markers for the merge conflict, namely the lines: HEAD === Function for overriding broken spanners If the result is the intended result, sure. You are getting the merge conflict because the history introduced

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
OK, I succeeded in doing this and uploaded a new patch set. I'm going to redo this now as I see I failed to delete two newlines... OK, all should be well now. Patch corrected. -David ___ lilypond-devel mailing list lilypond-devel@gnu.org

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
use it as a reviewing tool, not a patch management tool. David Nalesnik should send you the file he gets from git format-patch, after he fixed all merge conflicts etc. Arggh...sorry for making a hash of this :( Hopefully, this patch will be properly formatted... -David 0001-Function

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
David, On Tue, Jul 24, 2012 at 11:50 AM, David Kastrup d...@gnu.org wrote: David Nalesnik david.nales...@gmail.com writes: On Tue, Jul 24, 2012 at 11:34 AM, Graham Percival gra...@percival-music.ca wrote: On Tue, Jul 24, 2012 at 05:26:01PM +0100, Phil Holmes wrote: I'd

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
On Tue, Jul 24, 2012 at 12:05 PM, David Kastrup d...@gnu.org wrote: David Nalesnik david.nales...@gmail.com writes: Arggh...sorry for making a hash of this :( By the way, you aren't. You are dealing with flexible and powerful tools in action, and keeping up surprisingly well. That's

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
Phil, On Tue, Jul 24, 2012 at 12:07 PM, Phil Holmes m...@philholmes.net wrote: ** Pushed as f7085cf9b2ff111b7d30c8a59e367c771a7e3c52http://git.savannah.gnu.org/cgit/lilypond.git/commit/?h=stagingid=f7085cf9b2ff111b7d30c8a59e367c771a7e3c52. Patchy is now running. Thank you very much for

Re: push patch for issue 2679

2012-07-24 Thread David Nalesnik
On Tue, Jul 24, 2012 at 12:57 PM, David Kastrup d...@gnu.org wrote: Phil Holmes m...@philholmes.net writes: BTW - apologies for top-posting and that, but my Windows email system takes your mails as HTML and does this with them. His mails contain HTML and plain text as alternatives, and

installing needed file within LilyDev

2012-09-09 Thread David Nalesnik
Hi all, I'm trying to compile the latest master within LilyDev, and I'm running into a problem when I try to prepare the build directory. I get the following error: ERROR: Please install required programs: lh CTAN package (texlive-lang-cyrillic or texlive-texmf-fonts) Can someone give me

Re: installing needed file within LilyDev

2012-09-09 Thread David Nalesnik
Phil, On Sun, Sep 9, 2012 at 10:24 AM, Phil Holmes m...@philholmes.net wrote: I suspect this might be relevant: http://code.google.com/p/lilypond/issues/detail?id=2742 Piece of cake! Thank you very much, David ___ lilypond-devel mailing list

Re: [GLISS] turn xxx.yyy into (xxx yyy)

2012-09-12 Thread David Nalesnik
On Wed, Sep 12, 2012 at 3:38 AM, David Kastrup d...@gnu.org wrote: [...] Basically, if a music function wants to provide a shorthand for an override, not being able to specify an optional context is a nuisance. I currently have just the same problem writing a \hide function that is supposed

Re: Doc: improve documentation of Bézier curves (2858) (issue 6561064)

2012-09-28 Thread david . nalesnik
Trevor-- Thank you so much for taking this on! I've been pecking at documenting this for awhile, but got hung up on finding the perfect examples... What you have is clearer than what I cam up with. https://codereview.appspot.com/6561064/diff/1/Documentation/notation/changing-defaults.itely

Allow \shape to tweak music, swap its arguments (issue 6585052)

2012-10-02 Thread david . nalesnik
LGTM https://codereview.appspot.com/6585052/diff/1/input/regression/shape-other-curves.ly File input/regression/shape-other-curves.ly (right): https://codereview.appspot.com/6585052/diff/1/input/regression/shape-other-curves.ly#newcode18 input/regression/shape-other-curves.ly:18: \override

Re: Allow \shape to tweak music, swap its arguments (issue 6585052)

2012-10-03 Thread david . nalesnik
On 2012/10/03 13:57:25, dak wrote: [...] To those suggestions let me answer with the famous answer to the question Mr Gandhi, what do you think of Western civilization?, namely with I would consider it a good idea. Oh, for just one zinger like that... So the best I think I can do at

Re: Allow \shape to tweak music, swap its arguments (issue 6585052)

2012-10-03 Thread david . nalesnik
On 2012/10/03 15:17:26, dak wrote: [...] and I am currently immersed in parser work to make the Context.GrobName thing fly. Thank you very much for working on this! I am looking forward to being able to fix the mess at the beginning of \alterBroken (and a general offsetting function I

Re: Doc: improve documentation of Bézier curves (2858) (issue 6561064)

2012-10-06 Thread david . nalesnik
Trevor-- This looks great to me. I like the reordering, and I think the explanations are very clear. I've tweaked the curves a little in the examples (do what you like with my suggestions!), but otherwise: LGTM.

Re: Doc: improve documentation of Bézier curves (2858) (issue 6561064)

2012-10-07 Thread david . nalesnik
LGTM. And thanks again for doing this! -David https://codereview.appspot.com/6561064/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: LilyPond 2.17.4 released

2012-10-08 Thread David Nalesnik
On Mon, Oct 8, 2012 at 7:19 AM, Joseph Rushton Wakeling joseph.wakel...@webdrake.net wrote: - This version contains work in progress. You will have access to the very latest features, but some may be incomplete, and you may encounter bugs and crashes. If you

Re: Still alive

2012-10-11 Thread David Nalesnik
On Thu, Oct 11, 2012 at 6:11 PM, Valentin Villenave valen...@villenave.net wrote: On Tue, Oct 9, 2012 at 8:19 AM, m...@mikesolomon.org m...@mikesolomon.org wrote: Just a quick ping to let you know that I'm not dead - I've been swamped w/ work recently and just got engaged so I'm planning out

Re: Naming _another_ lacking puzzle piece

2012-10-13 Thread David Nalesnik
On Sat, Oct 13, 2012 at 8:15 AM, David Kastrup d...@gnu.org wrote: David Nalesnik david.nales...@gmail.com writes: Buildup of unwanted data or no, it would be useful to be able to write something like \toLast Accidental #'color to restore the previous override But which is the previous

tuplet number on cross-staff kneed-beam

2011-07-04 Thread David Nalesnik
Hello, all -- First of all, I hope that I'm asking this question on the appropriate list! I'm trying to simplify the workaround relating to tuplet-number position on kneed beams http://lsr.dsi.unimi.it/LSR/Snippet?id=646 and I'm running into an unexpected problem. My reasoning is that, since

Re: tuplet number on cross-staff kneed-beam

2011-07-05 Thread David Nalesnik
Hi, Colin -- On 7/5/11, Colin Campbell c...@shaw.ca wrote: This will probably require an issue, David, but could you send the code you used to produce the 2.12.3 version, please? I can't reproduce the previous stable example, in order to confirm the regression, because your Scheme-ing gives

Re: tuplet number on cross-staff kneed-beam

2011-07-05 Thread David Nalesnik
Hi, again -- I've also attached a pared-down version of the file which should work in either release, as is. It distills the problem a little more -- at the expense of centering the number directly on the beam . . . My last email included an image produced with 2.12.3. Attached is the

text on analysis brackets

2011-07-08 Thread David Nalesnik
Hi, all -- I'd like to be able to add text to analysis brackets, and I'm running into a problem with nested brackets. In the attached file, I'd like the last bracket to have b over it, but it takes its text from the previous incomplete bracket. (It seems I can add as many nested brackets as I

Re: text on analysis brackets

2011-07-08 Thread David Nalesnik
On 7/8/11, David Nalesnik dnale...@umail.iu.edu wrote: Hi, all -- I'd like to be able to add text to analysis brackets, and I'm running into a problem with nested brackets. In the attached file, I'd like the last bracket to have b over it, but it takes its text from the previous incomplete

Re: center NoteColumn

2011-12-04 Thread David Nalesnik
Hi Harm, On Tue, Nov 29, 2011 at 5:16 PM, Thomas Morley thomasmorle...@googlemail.com wrote: the attached code is my attempt to center a NoteColumn in a measure (like MultiMeasureRest). This is really cool! While it works fine in most cases, there is one major problem: Having a

Re: center NoteColumn

2011-12-04 Thread David Nalesnik
Hi again. On Fri, Dec 2, 2011 at 4:31 PM, David Nalesnik david.nales...@gmail.comwrote: (See attached file.) Oops--that only accidentally works for your problem example since I goofed the filtering. Replace the second definition of lst-2 in the file I last attached with this line and all

Re: center NoteColumn

2011-12-06 Thread David Nalesnik
Hi Harm, On Fri, Dec 2, 2011 at 7:06 PM, Thomas Morley thomasmorle...@googlemail.com wrote: But now there's a new problem. Sometimes I've to notice a very small, but visible displacement of the NoteColumn, if KeyCancellation is left bound and a new Clef is right bound (mes. 5 of the full

Re: center NoteColumn

2011-12-08 Thread David Nalesnik
Hi Harm, I've taken a look at your calculations and I think I've found a more straightforward way to determine the offsets for each of the elements. Rather than centering each element first, then moving it with an additional offset as you do, why not simply move everything from its _original_

Re: center NoteColumn

2011-12-08 Thread David Nalesnik
Hi Harm, On Thu, Dec 8, 2011 at 6:50 PM, Thomas Morley thomasmorle...@googlemail.com wrote: this is great !! And I was not a little but a big blockhead.;) Hah, no... I'm just a fresh pair of eyes :) Speaking of which, looking with fresh eyes at the file I just attached, I really should have

Re: center NoteColumn

2011-12-11 Thread David Nalesnik
Hi Harm, On Sat, Dec 10, 2011 at 3:42 PM, Thomas Morley thomasmorle...@googlemail.com wrote: Hi David, thanks for doing this! I did the next step to simplify the definition, with defining read-out. Yes, this condenses the function quite a bit. Thinking about the problem some more, I came

problem uploading a patch

2012-10-16 Thread David Nalesnik
Hi, I'm trying to upload a patch created in a branch to Rietveld, but I seem to be creating an unholy mess. I attempted to run git-cl upload master within my branch, but I end up with a huge list of recent commits not my own. The same thing happened when I ran git format-patch master and

Re: problem uploading a patch

2012-10-16 Thread David Nalesnik
Hi Phil, On Tue, Oct 16, 2012 at 3:11 PM, Phil Holmes [...] If you have a record of changes you've made, then I tend to use git reset --hard origin/master which takes you back to the current state of master, and then git am your.patch which applies the patch you have. Follow this

Re: problem uploading a patch

2012-10-16 Thread David Nalesnik
Hi Phil. david@david-desktop ~/lilypond-git (dev/measure_counter)$ git cl issue 2445 Issue number: 2445 (http://codereview.appspot.com/2445) david@david-desktop ~/lilypond-git (dev/measure_counter)$ git cl upload Unable to determine default branch to diff against. Either pass complete git

Re: problem uploading a patch

2012-10-17 Thread David Nalesnik
David, On Wed, Oct 17, 2012 at 12:44 AM, David Kastrup d...@gnu.org wrote: Try git rebase origin first I tried this, and this is what happened: david@david-desktop ~/lilypond-git (dev/measure_counter)$ git rebase origin fatal: Needed a single revision invalid upstream origin Should I try

Re: problem uploading a patch

2012-10-17 Thread David Nalesnik
Julien, On Wed, Oct 17, 2012 at 1:33 AM, Julien Rioux jri...@physics.utoronto.ca wrote: On 16/10/2012 4:36 PM, David Nalesnik wrote: david@david-desktop ~/lilypond-git (dev/measure_counter)$ git cl issue 2445 Issue number: 2445 (http://codereview.appspot.com/2445) This should be the issue

Issue 2445: Add measure counter to LilyPond (issue 6730044)

2012-10-17 Thread david . nalesnik
Reviewers: , Message: Please review. Thanks! -David Description: Issue 2445: Add measure counter to LilyPond This patch allows the user to number groups of successive measures, which is useful as an aid to the performer in keeping track of repetitions. There is no requirement that the

Re: problem uploading a patch

2012-10-17 Thread David Nalesnik
David, On Wed, Oct 17, 2012 at 8:04 AM, David Kastrup d...@gnu.org wrote: David Nalesnik david.nales...@gmail.com writes: Julien, On Wed, Oct 17, 2012 at 1:33 AM, Julien Rioux jri...@physics.utoronto.ca wrote: On 16/10/2012 4:36 PM, David Nalesnik wrote: david@david-desktop ~/lilypond

Re: problem uploading a patch

2012-10-17 Thread David Nalesnik
Julien, On Wed, Oct 17, 2012 at 9:25 AM, Julien Rioux jri...@physics.utoronto.ca wrote: $ git cl issue 0 to reset (remove any association to a Rietveld issue) $ git cl upload origin/master to upload (given that the branch is no longer associated to a Rietveld issue, it will create a new

Re: Issue 2445: Add measure counter to LilyPond (issue 6730044)

2012-10-24 Thread david . nalesnik
Thanks for your review, Janek! https://codereview.appspot.com/6730044/diff/10001/input/regression/measure-counter-broken.ly File input/regression/measure-counter-broken.ly (right): https://codereview.appspot.com/6730044/diff/10001/input/regression/measure-counter-broken.ly#newcode6

Re: Issue 2445: Add measure counter to LilyPond (issue 6730044)

2012-10-25 Thread david . nalesnik
On 2012/10/24 20:28:36, janek wrote: On Wed, Oct 24, 2012 at 4:10 PM, mailto:pkx1...@gmail.com wrote: Also, should we include Measure_counter_engraver in the Staff context by default? (it'd make documenting it simpler in the @lilypond if nothing else :) ) +1 from me :) Janek

Change \alterBroken interface to match that of other tweak/overrides (issue 6784063)

2012-10-28 Thread david . nalesnik
LGTM! Thanks very much for doing this, David. https://codereview.appspot.com/6784063/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Issue 2445: Add measure counter to LilyPond (issue 6730044)

2012-10-31 Thread david . nalesnik
David, I don't mind making the changes, but I'm running into some problems. I used git pull -r before making a patch set and uploading to rietveld, but looking over the patch set there I see that other changes have found their way in--including your check-grob-path function. I'm probably

Re: Issue 2445: Add measure counter to LilyPond (issue 6730044)

2012-10-31 Thread david . nalesnik
Thank you--I am a bit worried! Speaking of patch, when I run git format-patch I get six separate patches. Is there any way to compress them into one so this is more convenient for you? -David https://codereview.appspot.com/6730044/ ___

Re: new markup-comand `draw-dashed-line´ was: lilypond-user Digest, Vol 119, Issue 90

2012-11-03 Thread David Nalesnik
Hi Harm, On Thu, Nov 1, 2012 at 9:57 AM, Thomas Morley thomasmorle...@googlemail.com wrote: Hi David (N), attached my first try. If the markup-property full-length is set #t, some calculations ensure that no space occurs at line-end/begin. Some dashes will only be partly visible, but I

Re: Aleatoric / modern notation

2012-12-08 Thread David Nalesnik
Hi Jeffrey, I'm copying this to the list since I myself am not sure about something here. On Sat, Dec 8, 2012 at 5:13 AM, Jeffrey Trevino jeffrey.trevi...@gmail.com wrote: Hi David, Thanks for revising this more for me. I really appreciate your efforts, and I will take a look at this asap

Re: Aleatoric / modern notation

2012-12-08 Thread David Nalesnik
Hi again, On Sat, Dec 8, 2012 at 7:21 AM, David Nalesnik david.nales...@gmail.com wrote: Hi Jeffrey, I'm copying this to the list since I myself am not sure about something here. On Sat, Dec 8, 2012 at 5:13 AM, Jeffrey Trevino jeffrey.trevi...@gmail.com wrote: Hi David, Thanks

Re: Aleatoric / modern notation

2012-12-10 Thread David Nalesnik
David, On Sun, Dec 9, 2012 at 12:34 AM, David Kastrup d...@gnu.org wrote: David Nalesnik david.nales...@gmail.com writes: It turns out that the my definition of event-drul as '(() . ()) was the problem. I substituted (cons '() '()) and everything works just fine...even with the file

Re: Aleatoric / modern notation

2012-12-11 Thread David Nalesnik
On Mon, Dec 10, 2012 at 11:30 PM, David Kastrup d...@gnu.org wrote: The behavior is undefined if I remember correctly: an error is not guaranteed by the standard. I think Guilev2 will produce an error, but LilyPond is still at Guilev1. OK, thanks! -David

Re: Aleatoric / modern notation

2012-12-11 Thread David Nalesnik
On Sat, Dec 8, 2012 at 7:21 PM, David Nalesnik david.nales...@gmail.com wrote: [...] Trying to work out the problem, I've done a little rewriting so I'll include the engraver here. You can just substitute this in the problematic file. Well, I did break something. Here's the fix

Re: Allows for easier creation of many Lilypond objects via Scheme. (issue 7009047)

2012-12-24 Thread David Nalesnik
On Mon, Dec 24, 2012 at 7:22 AM, m...@mikesolomon.org m...@mikesolomon.org wrote: On 24 déc. 2012, at 10:36, d...@gnu.org wrote: On 2012/12/24 07:28:17, mike7 wrote: On 24 déc. 2012, at 01:10, mailto:d...@gnu.org wrote: All of this is absolutely devastatingly horrible code that is not

Re: Introducing two new markup-commands: draw-dashed-line and draw-dotted-line. (issue 7029045)

2012-12-31 Thread david . nalesnik
Harm-- This looks great! Thank you for the 'full-length option. I can't be alone in hating lines ending with incomplete dashes. All I have is a suggestion or two, and some quibbles. Besides what I've pointed out inline, I should mention that I got a number of whitespace errors when I applied

Re: i'm back - hopefully.

2013-02-11 Thread David Nalesnik
2013/2/12 Janek Warchoł janek.lilyp...@gmail.com: Hi all, i hope to get back to lilypond hacking now. I miss it a lot... Welcome back, Janek! --David ___ lilypond-devel mailing list lilypond-devel@gnu.org

make doc fails

2013-03-16 Thread David Nalesnik
Hi all, I'm attempting to put two snippets dealing with the measure counter up for review, but I'm running into problems when I run make doc. Here's the tail end of the terminal output: make[3]: Entering directory `/home/david/lilypond-git/build/Documentation/cs' /usr/bin/python -tt

Issue 2924: Doc: \startMeasureCount and \stopMeasureCount needs documenting (issue 7678047)

2013-03-16 Thread david . nalesnik
Reviewers: , Message: Please review. Thanks! David Description: Issue 2924: Doc: \startMeasureCount and \stopMeasureCount needs documenting The following patch is a preliminary step towards documenting the Measure_counter_engraver added in version 2.17.7. The patch consists of two snippets,

Re: make doc fails

2013-03-16 Thread David Nalesnik
Julien, On Sat, Mar 16, 2013 at 7:31 PM, Julien Rioux jri...@lyx.org wrote: On 16/03/2013 6:27 PM, David Nalesnik wrote: /usr/bin/python -tt /home/david/lilypond-git/**scripts/build/create-weblinks-**itexi.py out-www/weblinks.itexi make[3]: *** No rule to make target `/home/david

Re: Issue 2924: Doc: \startMeasureCount and \stopMeasureCount needs documenting (issue 7678047)

2013-03-17 Thread david . nalesnik
On 2013/03/17 10:35:40, PhilEHolmes wrote: I've not tested the snippets, but otherwise the patch LGTM. I'm assuming the only files you actually created were the ones in snippets/new/ - the rest came from makelsr and make doc? Yes, exactly. https://codereview.appspot.com/7678047/

Re: PATCHES: Countdown - for March 26th 19:00GMT

2013-03-23 Thread David Nalesnik
%20Patch%20Needs%20Summary%20Modified Documentation David Nalesnik Push Doc: \startMeasureCount and \stopMeasureCount needs documenting I don't have push privileges--could someone push the patch for issue 2924 for me? Thanks, David ___ lilypond

Re: Issue 2924: Doc: \startMeasureCount and \stopMeasureCount needs documenting (issue 7678047)

2013-03-23 Thread david . nalesnik
Thank you all for the reviews! https://codereview.appspot.com/7678047/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Add the command \offset to LilyPond (issue 8647044)

2013-04-13 Thread david . nalesnik
Reviewers: , Message: Please review. Thanks, David Description: Add the command \offset to LilyPond The ability to offset default values of various properties would be a useful enhancement of LilyPond. Currently, this is possible for the property 'control-points using the \shape command.

Re: Add the command \offset to LilyPond (issue 8647044)

2013-04-16 Thread david . nalesnik
I've considerably shortened the regtests, removing redundancies and unnecessary line breaks. Thanks for the comments! https://codereview.appspot.com/8647044/diff/1/input/regression/offsets.ly File input/regression/offsets.ly (right):

  1   2   3   4   5   >