Re: Beautify Grob_array and stop using std::vector::data() (issue 264950043 by nine.fierce.ball...@gmail.com)

On Sat, 05 Sep 2015 17:49:44 -0700, wrote: I have simply removed the comments warning about filtering ordered arrays. That seems to be correct. Those comments were incorrect. The filter_map* functions do not, in fact, change the order of the array under the

Re: Doc: Issue 4059: Document MIDI mapping and MIDI effects (issue 249980043 by tdanielsmu...@googlemail.com)

On Mon, 06 Jul 2015 15:19:20 -0700, tdanielsmu...@googlemail.com wrote: Setting midiChannelMapping to 'voice often prevents the context properties from having effect. Doesn't the paragraph under 'voice above explain explicitly what happens? That seems better than the rather vague often

Re: Avoid treating x as relative in \relative { \autochange {x} } (issue 250170043 by nine.fierce.ball...@gmail.com)

On Sat, 04 Jul 2015 05:45:04 -0700, nine.fierce.ball...@gmail.com wrote: Nobody knew \autochange was broken until Thomas found an unused variable, so these test are useful if \autochange is useful. Well, \autochange could have been useful, even to users of \relative if they used their

Re: Make music functions callable from Scheme (issue 244840043 by d...@gnu.org)

On Sat, 20 Jun 2015 22:28:13 -0700, David Kastrup d...@gnu.org wrote: Keith OHara k-ohara5...@oco.net writes: Some people don't like side-effects in loop-control conditionals while (scm_is_pair (scm_cdr (signature)) // more slots ? scm_is_pair (scm_cadr (signature)) // next

Re: Make music functions callable from Scheme (issue 244840043 by d...@gnu.org)

On Mon, 08 Jun 2015 18:04:44 -0700, d...@gnu.org wrote: No dice. The main problem is dealing with all combinations of running out of signature and running out of available arguments. If you treat the optional argument skipping in an inner loop, you may still run out of signature, and in that

Re: balloon-text: robust with empty stencils; issue 4447 (issue 243610043 by k-ohara5...@oco.net)

On Fri, 19 Jun 2015 23:50:50 -0700, d...@gnu.org wrote: https://codereview.appspot.com/243610043/diff/1/lily/balloon.cc#newcode94 lily/balloon.cc:94: b.widen (padding, padding); Basically we have two options here: not draw a box at all (it would be my guess that this is probably what happened

Re: simple-spacer: inappropriate assert; issue 4448 (issue 249820043 by k-ohara5...@oco.net)

On Fri, 19 Jun 2015 23:59:11 -0700, d...@gnu.org wrote: https://codereview.appspot.com/249820043/diff/1/lily/simple-spacer.cc File lily/simple-spacer.cc (right): https://codereview.appspot.com/249820043/diff/1/lily/simple-spacer.cc#newcode199 lily/simple-spacer.cc:199: programming_error

Re: Make music functions callable from Scheme (issue 244840043 by d...@gnu.org)

On Sun, 07 Jun 2015 23:09:31 -0700, d...@gnu.org wrote: So your problem appears to be that you understand the code but it does something you did not expect. Cross-check from the user end with the Extending LilyPond guide's explanation of optional arguments. It is Extending section 2.2.2.

Re: add stencil-whiteout-outline function (issue 236480043 by paulwmor...@gmail.com)

On Mon, 01 Jun 2015 19:29:41 -0700, paulwmor...@gmail.com wrote: scripts/auxiliar/update-with-convert-ly.sh I get this: make: *** No rule to make target `python-modules'. Stop. xargs: ./out/bin/convert-ly: No such file or directory xargs: ./out/bin/convert-ly: No such file or directory I'm

Re: git-cl upload has stopped working (so has patchy)

Trevor Daniels t.daniels at treda.co.uk writes: The next step is to decide our preference: do we want to take on the commitment to maintain Allura ourselves or use the maintained version at SourceForge. Which would we prefer? (FWIW my preference would be to run Allura at Savannah as our

Re: Master fails to compile

On Thu, 28 May 2015 01:36:54 -0700, David Kastrup d...@gnu.org wrote: Keith OHara k-ohara5...@oco.net writes: I had an old RedHat installation with gcc updated as far as I could to 4.6. In the next few weeks I'll find some reasonably-modern linux. Well, using no longer supported systems

Re: Master fails to compile

On Mon, 25 May 2015 23:10:17 -0700, David Kastrup d...@gnu.org wrote: Keith OHara k-ohara5...@oco.net writes: If I can't get the dependencies worked out I'll be tempted to skip the macro and use scm_call_1() through scm_call_3() in place of each of the three macro expansions. Why

Re: Master fails to compile

Phil Holmes mail at philholmes.net writes: I've managed to work out how to upgrade Ubuntu 10.04 to gcc 4.8.1 with the help of StackOverflow, and can now compile again. I'm still working on sorting out dependencies. The troublesome macro asks is used in only three places, creating in each

Re: PDF is broken for at notation{} encoding

Phil Holmes mail at philholmes.net writes: whenever we use at notation{text}, then there is a paragraph break before the text. It took a while to figure out the guilty commit, but it is 9f3c7711bb73baf3aea8502227fb5fd2d2851753: update texinfo.tex from upstream. I think you

Re: pure/impure?!?!

On Sun, 24 May 2015 01:55:30 -0700, David Kastrup d...@gnu.org wrote: Anyway: how do impure functions reference (and trigger?) line breaks? Non-'pure' functions reference line-breaks very indirectly. Spanners are broken into two or more grobs at line-breaking, so their functions work on the

Re: absolute pitch entry: accept an offset octave (issue 235010043 by k-ohara5a5a at oco.net)

Wols Lists antlists at youngman.org.uk writes: On 23/05/15 23:39, Keith OHara wrote: tdanielsmusic at googlemail.com writes: I'd prefer the syntax and options to parallel those of \relative. That is, an optional prefix pitch to indicate the starting octave, and taking the starting

Re: absolute pitch entry: accept an offset octave (issue 235010043 by k-ohara5a5a at oco.net)

Wols Lists antlists at youngman.org.uk writes: On 21/05/15 07:36, k-ohara5a5a at oco.net wrote: I'm proposing to push a \fixed c' {} that always takes a reference pitch, as in the current patch. So the behaviour is different from \relative, which doesn't have to have a reference pitch.

Re: pure/impure?!?!

David Kastrup dak at gnu.org writes: Ignored or not: those are values indicating breakpoints of the current system. They should not even be available for functions evaluated before any line-breaking. Or maybe they are called for one of a set of tentative not-final line breaks, but in that

Re: Add French-specific note names (issue 239930043 byv.villenave at gmail.com)

Trevor Daniels t.daniels at treda.co.uk writes: BTW dear lilylist: where do you have the 'ñ' on your keyboards? We don't have it anywhere on the English keyboard. You might have more than you think, or can easily get more. The US-International layout has the accented characters of

Re: absolute pitch entry: accept an offset octave (issue 235010043 by k-ohara5...@oco.net)

On Sun, 17 May 2015 04:58:22 -0700, tdanielsmu...@googlemail.com wrote: The proper name for this would \absoluteWithFixedOctaveOffset, but that's too long and the acronym is similarly uninspiring. All three of the proposed options appear in this name, so the question is, Which alludes most

Re: Make \shiftOff an assertive \override, not a \revert (issue 190500043 by k-ohara5...@oco.net)

On Sun, 10 May 2015 11:02:21 -0700, d...@gnu.org wrote: It's not clear why you are now mixing in variants of conversion rules for issue 4372. I noticed that my rule made the same \b mistake as yours, and they are right next to each other, and I thought it was an easy fix, so I tried fixing

Re: absolute pitch entry: accept an offset octave (issue 235010043 by k-ohara5...@oco.net)

On Sun, 10 May 2015 00:25:18 -0700, d...@gnu.org wrote: On 2015/05/09 23:58:22, Keith wrote: It is a bit of an inconvenience that \relative skips over \transpose, and I suspect \relative has this feature so that we do not get rising octaves in \relative c \transpose c c' {c c c c} We

Re: Make \shiftOff an assertive \override, not a \revert (issue 190500043 by k-ohara5...@oco.net)

On Sun, 10 May 2015 05:06:24 -0700, d...@gnu.org wrote: In addition, the following commit commit a1d73a332fc4d1cb99bace19a5ec2c35262429e7 Author: Keith OHara k-ohara5...@oco.net Date: Tue Dec 30 23:30:27 2014 -0800 Update by hand instead of convert-ly The documentation already

Re: absolute pitch entry: accept an offset octave (issue 235010043 by k-ohara5...@oco.net)

On Mon, 04 May 2015 23:58:12 -0700, Valentin Villenave valen...@villenave.net wrote: Speaking as someone who regularly gives LilyPond initiation seminars for adults and children, the hardest part is explaining to them why \relative mode is not on by default. What if they didn't have to type

Re: \change Voice

On Sun, 26 Apr 2015 14:35:33 -0700, Dan Eble d...@faithful.be wrote: If there existed a command to redirect the sequential iterator’s output to another Voice context without requiring a wrapper context, it could not function in a parallel sequence; the commands would have to be injected into

Re: \change Voice

On Sun, 26 Apr 2015 05:37:14 -0700, Dan Eble d...@faithful.be wrote: I think \context...{} segments make a better implementation of \partcombine than re-routing outlets in the part-combine-iterator.cc I believe that would work, but it looks less tidy than keeping the original part intact

Re: \change Voice

On Fri, 24 Apr 2015 22:04:28 -0700, David Kastrup d...@gnu.org wrote: Keith OHara k-ohara5...@oco.net writes: The thing you want to change is the outlet of the sequential iterator. That's just \context Voice = xxx. However, it does not take engravers and properties along, as opposed

Re: \change Voice

Dan Eble dan at faithful.be writes: Would it be hard to make \change Voice work as below without relying on the extra “NoteSequence” context? \layout { \context { \name Thread \type Engraver_group } \context { \Voice \accepts Thread \defaultchild Thread

Re: Avoid floating-point compares on Stem:head_positions; issue 4310 (issue 217720043 by k-ohara5...@oco.net)

On Wed, 11 Mar 2015 02:10:29 -0700, d...@gnu.org wrote: _How_ do we arrive at different versions of 3.0 here, or at GCC considering 3.0 3.0 ? Probably we get different numbers that are close to 3.00... or 2.99... The head positions are individually multiplied by staff-space in

Re: Lilypond build: version

Phil Holmes mail at philholmes.net writes: I completed a full GUB build of lilypond yesterday That's very good news. I was beginning to worry. Thanks for taking the time to work through the problems. ___ lilypond-devel mailing list

Re: Issue 4275: Allow user-defined rest styles. (issue 200860043 by nine.fierce.ball...@gmail.com)

On Sat, 31 Jan 2015 14:49:29 -0800, nine.fierce.ball...@gmail.com wrote: https://codereview.appspot.com/200860043/diff/20001/lily/rest.cc#newcode166 lily/rest.cc:166: + offset); On 2015/01/31 06:02:50, Keith wrote: I guess that get_position() could possibly trigger a full layout, but I can't

Re: Issue 4274: Fix a cyclic dependency in Rest_collision (issue 193590043 by nine.fierce.ball...@gmail.com)

On Wed, 28 Jan 2015 17:40:33 -0800, nine.fierce.ball...@gmail.com wrote: Yes, the ledgered rest stuff is a big tangle. See https://codereview.appspot.com/200860043/ for the best I've been able to do without changing Rest_collision. Shucks. The similar issue with Accidentals (issue 3646) was

Re: Issue 4274: Fix a cyclic dependency in Rest_collision (issue 193590043 by nine.fierce.ball...@gmail.com)

On Tue, 27 Jan 2015 05:09:58 -0800, nine.fierce.ball...@gmail.com wrote: On 2015/01/27 05:21:47, Keith wrote: I discovered this when I tried to use get_property(stencil) from Rest::height while trying to fix issue 4245. It looks like some rests (whole and half rests) have two glyphs, with

Re: Reduce size of PDF files when inc. in *TeX docs (issue 194090043 by pkx1...@gmail.com)

On Tue, 20 Jan 2015 13:19:26 -0800, pkx1...@gmail.com wrote: https://codereview.appspot.com/194090043/diff/20001/ps/encodingdefs.ps#newcode8 ps/encodingdefs.ps:8: /LilyNoteHeadEncoding [ On 2015/01/18 06:33:02, Keith wrote: This is a little different from FetaNoteheadsEncoding in

Re: mentorship

Kevin Barry barrykp at gmail.com writes: Thank you for responding. I would like to be able to contribute code to the project but I'm not really a programmer (I'm a music academic). I worked through some chapters of SICP in the past, so I have a rudimentary understanding of ... the first few

Re: GUB fail with smob templates

Phil Holmes mail at philholmes.net writes: Phil Holmes mail at philholmes.net writes: /home/gub/gub/target/darwin-ppc/src/lilypond-git.sv.gnu.org-- lilypond.git- release-unstable/lily/include/smobs.tcc:131: error: invalid operands of types 'scm_unused_struct* (Grob::*)()' and

Re: crosslinks update

Jean-Charles Malahieude lilyfan at orange.fr writes: I've locally merged master into translation, and before merging translation back to staging will correct bad cross-links. Sorry, I should have changed the rlearning links. Don't forget that translators use the English node names with

Re: Reduce size of *TeX generated pdfs with lilypond snippets

Knut Petersen Knut_Petersen at t-online.de writes: The attached patch (against 2.18.2) changes the way lilypond uses fonts to draw glyphs. It avoids to used glyphshow for all emmentaler glyphs and adds encoding vectors to the emmentaler fonts before they are used. It also changes the

Re: Reduce size of *TeX generated pdfs with lilypond snippets

On Sun, 28 Dec 2014 21:10:59 -0800, Werner LEMBERG w...@gnu.org wrote: https://codereview.appspot.com/191360043/ Unfortunately, this doesn't correctly include the new file `encodingdefs.ps' – it is tagged as a `binary'... Oops, forgot to change the settings. There is no existing file to

Re: GUB and mpfr/mpc

Dan Eble dan at faithful.be writes: On Dec 8, 2014, at 08:21 , David Kastrup dak at gnu.org wrote: Masamichi HOSODA trueroad at sea.plala.or.jp writes: I agree that changing the algorithms is preferred; I didn’t mean to suggest otherwise. But if that’s not going to happen

Re: mentorship

Kevin Barry barrykp at gmail.com writes: It depends on which part of LilyPond you want to help with as to the availability of people to help you. Is there a particular skill-set, set of issues or area of LilyPond you are particularly interested in? Thank you for responding. I would

Re: Part combiner: separate split state and voice names

Urs Liska ul at openlilylib.org writes: Keith wrote: I suggest looking over the existing partcombine bugs, and orchestral scores, to see what problems generally need solving. If there's anything I can do to help (without understanding more than basic Scheme and without any option to

Re: Part combiner: separate split state and voice names

On Sun, 30 Nov 2014 13:46:35 -0800, Dan Eble d...@faithful.be wrote: I have a question. If the Scheme code produces something like (‘apart “one” “two”) with “one” and “two” being the chosen output voices for the input parts at the moment, would it make sense to write those decisions back to

Re: Part combiner: separate split state and voice names

Dan Eble dan at faithful.be writes: I would like to break the rigid link between the split states (apart, unisilence, etc.) and the names of the voices into which the Part_combine_iterator routes events. The patch back at

Re: Part combiner: separate split state and voice names

On Sat, 29 Nov 2014 14:44:48 -0800, Dan Eble d...@faithful.be wrote: On Nov 29, 2014, at 16:46 , Keith OHara k-ohara5...@oco.net wrote: Dan Eble dan at faithful.be writes: I would like the Scheme portion of the part combiner to dictate the names of the voices to use at every moment

Re: Part combiner: separate split state and voice names

On Sat, 29 Nov 2014 18:09:34 -0800, Dan Eble d...@faithful.be wrote: On Nov 29, 2014, at 18:48 , Keith OHara k-ohara5...@oco.net wrote: Oh, so you meant let the Scheme portion dictate to partcombine_iterator which _input_ voice, as it iterates the music expression produced by \partcombine

Re: Add chord range to make-part-combine-music (issue 144170043 by nine.fierce.ball...@gmail.com)

On Fri, 07 Nov 2014 04:37:41 -0800, nine.fierce.ball...@gmail.com wrote: On 2014/11/06 05:44:50, Keith wrote: Changing the current global variable 'partcombine-chord-range' will not become the user interface. All I meant for that variable was to avoid repeating the same magic numbers three

Re: Issue 3286: add single-C time signature style (issue 164830043 by nine.fierce.ballads at gmail.com)

On Mon, 03 Nov 2014 05:11:02 -0800, Dan Eble d...@faithful.be wrote: On Nov 3, 2014, at 02:25 , Keith OHara k-ohara5...@oco.net wrote: That kind logic is already at the end of Time_signature::special_time_signature() now, but I don't see any way to use it for anything except 2/2 and 4/4

Re: how to make space between time signature and following material squeezable

This is https://code.google.com/p/lilypond/issues/detail?id=1785 For four years this space was too compressible, and would not stretch. The behavior did not match the names of the settings and code comments, so I raised issue 1785 to make the space rigid. Another option, semi-fixed-space in

Re: staff-symbol-referencer: ledger logic; issue 4184 (issue 167040043 by k-ohara5...@oco.net)

'notehead' noted. On Sat, 01 Nov 2014 23:43:34 -0700, lemzw...@googlemail.com wrote: /* If we are closer to the staff than the notehead, quantize for ledger lines. */ But I wonder whether this is the best description – it's rather about note heads and attached grobs having the same

Re: Issue 3286: add single-C time signature style (issue 164830043 by nine.fierce.ballads at gmail.com)

Dan Eble dan at faithful.be writes: On Nov 1, 2014, at 18:48 , Noeck noeck.marburg at gmx.de wrote: * leave the default style alone * add to the C style: 4/2 - CC, 2/1 - cut-CC This would end the equivalence of default and C styles. Does that seem like a bad idea to any seasoned

Re: Issue 3286: add single-C time signature style (issue 164830043 by nine.fierce.ballads at gmail.com)

On Sun, 02 Nov 2014 18:11:58 -0800, Dan Eble d...@faithful.be wrote: On Nov 2, 2014, at 14:52 , Keith OHara k-ohara5...@oco.net wrote: The current code has some logic to choose a glyph timesig.C34 if... I followed everything you said except this, maybe because of the sudden mention of 3/4

Re: Add chord range to make-part-combine-music (issue 144170043 by nine.fierce.ballads at gmail.com)

dak at gnu.org writes: On 2014/10/31 22:50:35, Dan Eble wrote: One thing just occurred to me. If this option is going into the UI, do you think the numbers should be 1-based for easier comprehension by musicians? As a UI, numbers instead of intervals (possibly represented only by a

Re: Add chord range to make-part-combine-music (issue 144170043 by nine.fierce.ball...@gmail.com)

On Thu, 30 Oct 2014 20:19:01 -0700, nine.fierce.ball...@gmail.com wrote: This is the beginning of the function I've actually been using: (define-public (dfe-determine-split-list evl1 evl2 chord-range enable-solo) @var{evl1} and @var{evl2} should be ascending. @var{chord-range} is a pair

Re: Add chord range to make-part-combine-music (issue 144170043 by nine.fierce.ball...@gmail.com)

On Fri, 31 Oct 2014 15:50:35 -0700, nine.fierce.ball...@gmail.com wrote: One thing just occurred to me. If this option is going into the UI, do you think the numbers should be 1-based for easier comprehension by musicians? For example, use '(1 . 9) to allow chording unisons to ninths, rather

Re: Doc: Issue 3324: Add explanation of clashing note columns warning (issue 154640043 by tdanielsmu...@googlemail.com)

On Thu, 23 Oct 2014 02:43:56 -0700, tdanielsmu...@googlemail.com wrote: On 2014/10/22 05:58:54, Keith wrote: I have been thinking that when the user gives explicit \voiceOne or \shiftOn to the voices, Lily should set those voices at the shifts requested, even if two voices have the same

Re: Issue 4169: Line and page breaking syntactic sugar (issue 156400043 by tdanielsmu...@googlemail.com)

On Tue, 21 Oct 2014 09:33:56 -0700, tdanielsmu...@googlemail.com wrote: I've decided to keep the reverts for the On predefs, since \once does not do what [the name implies] with these overrides. Yuck. The problem seems to be that the override takes effect at the first beat in a bar, after

Re: Simplify the NullVoice context (issue 117050043 by k-ohara5...@oco.net)

On Mon, 08 Sep 2014 16:28:24 -0700, Dan Eble d...@faithful.be wrote: How about placing the voices inside a VoiceGroup context, moving the Accidental_engraver there, and leaving the NullVoice outside, like in my recent partcombine experiments? Usually, accidentals are reported up the

Re: Simplify the NullVoice context (issue 117050043 by k-ohara5...@oco.net)

On Sun, 07 Sep 2014 01:21:58 -0700, d...@gnu.org wrote: But messing up the basic grob definition by fetching internals and meddling through them locally: no. Well, you say messing up the basic grob definition I say reading the basic definition, and making a local modified copy We can do in

Re: Issue 4083: Implement \tagGroup command (issue 137920043 by d...@gnu.org)

Patch set 5 will probably be fine. My concern was that someone would be confused that adding to the list of tags to keep \keepWithTag editorial { ... \tag#'editorial \tag#'score \bar'||' } \keepWithTag part.editorial { ... \tag#'editorial \tag#'score \bar'||' } can reduce the things that

Re: Die, instrumentSwitch. (issue 133390043 by k-ohara5...@oco.net)

On Sun, 31 Aug 2014 01:06:46 -0700, tdanielsmu...@googlemail.com wrote: A few minor comments below, but I don't see how we can just remove the instructions for setting up InstrumentSwitch without some replacement. InstrumentSwitch is used by \cueDuring and friends. \cueDuring itself does not

Re: Issue 4078: Doc: Use variables rather than instrument definitions (issue 138950043 by tdanielsmu...@googlemail.com)

On Sun, 31 Aug 2014 00:14:11 -0700, d...@gnu.org wrote: On 2014/08/31 06:58:47, Keith wrote: https://codereview.appspot.com/138950043/diff/1/Documentation/notation/vocal.itely#newcode2646 \set instrumentCueName = Flute This is the *other* use of instrumentSwitch, which I'll probably put back

Re: indclude notnames bn, etc., in English (issue 133840043 by k-ohara5...@oco.net)

On Mon, 25 Aug 2014 00:28:14 -0700, pkx1...@gmail.com wrote: Excuse my ignorance, is this going to need any extra documentation or perhaps a changes.tely entry? https://codereview.appspot.com/133840043/ I read through the documentation and saw no changes except mentioning the alternative

Re: AccidentalCautionary in NullVoice

On Sat, 23 Aug 2014 13:16:22 -0700, Dan Eble d...@faithful.be wrote: On Aug 23, 2014, at 08:03 , Dan Eble d...@faithful.be wrote: I think I’m really close (again!) to getting modern-voice-cautionary accidentals with combined-part stems by defining a new accidental style that disregards the

Re: AccidentalCautionary in NullVoice

On Sat, 16 Aug 2014 11:29:29 -0700, Dan Eble d...@faithful.be wrote: On Aug 11, 2014, at 02:43 , Keith OHara k-ohara5...@oco.net wrote: I'll still put NullVoice in \score by default, so it doesn't interact with the accidental engraver, or anything else that we haven't noticed yet, and let

Re: AccidentalCautionary in NullVoice

On Sun, 10 Aug 2014 17:15:47 -0700, Dan Eble d...@faithful.be wrote: On Aug 10, 2014, at 15:40 , Dan Eble d...@faithful.be wrote: On Aug 10, 2014, at 14:55 , Keith OHara k-ohara5...@oco.net wrote: You could then enter \accepts NullVoice into just those Staffs where you use this technique

Re: AccidentalCautionary in NullVoice

On Sun, 10 Aug 2014 10:05:04 -0700, Dan Eble d...@faithful.be wrote: Patch set 3 seems more useful to me than patch set 4. Yep. I'll combine the best of both: 1) NullVoice in Score by default, so that the invisible notes do not affect the setting of accidentals 2) settings enabled so that

Re: Issue 4024: Clarify break-align symbols and space-alist args in IR. (issue 114160044 by markpole...@gmail.com)

On Tue, 29 Jul 2014 13:12:47 -0700, markpole...@gmail.com wrote: Ah, I see. Here's the confusion: when paired with right-edge, *all* 5 of the spacing-styles actually do something (to be clear, they all do the same thing: they behave like extra-space, with space that doesn't stretch). Anyway,

Re: Simplify the NullVoice context (issue 117050043 by k-ohara5...@oco.net)

On Sun, 27 Jul 2014 23:07:58 -0700, d...@gnu.org wrote: The ugliness is with messing up the whole grob definition of a NoteHead (which is fixed to the degree of being documented in the Internals Reference) inside of a particular context. I thought that was pretty, as well. We override the

Re: Set X-parent of TextScript to NoteColumn instead of PaperColumn (issue 106640043 by janek.lilyp...@gmail.com)

On Sun, 27 Jul 2014 09:44:22 -0700, janek.lilyp...@gmail.com wrote: On 2014/07/26 21:36:12, Keith wrote: On 2014/07/26 06:49:41, janek wrote: Setting TextScript.cross-staff property to #f is required to ensure that there are no collisions between TextScripts and cross-staff

Re: AccidentalCautionary in NullVoice

Dan Eble dan at faithful.be writes: On Feb 9, 2014, at 21:19 , Dan Eble dan at faithful.be wrote: Here is a work-around for https://code.google.com/p/lilypond/issues/detail?id=3834 NullVoice looks like a long list of workarounds. Now that Janek has independent Lyrics aligned as of

Re: AccidentalCautionary in NullVoice

Dan Eble dan at faithful.be writes: It sounds like you are saying the issue with spurious accidental cancellations is fixed in 2.19.9 No, NullVoice still has those problems, because it still creates invisible note-heads, and applies various workarounds so that the note-heads don't have any

Re: fix Issue 2462. (issue 108280044 by janek.lilyp...@gmail.com)

On Fri, 04 Jul 2014 14:38:19 -0700, Janek Warchoł janek.lilyp...@gmail.com wrote: 2014-07-04 23:26 GMT+02:00 Janek Warchoł janek.lilyp...@gmail.com: That's actually not what i want to do; I apologize for being unclear. I *want* the columns to behave the same way in case of both accidentals

Re: fix Issue 2462. (issue 108280044 by janek.lilyp...@gmail.com)

On Wed, 02 Jul 2014 23:56:20 -0700, janek.lilyp...@gmail.com wrote: you should try to make the change consistently everywhere ? Yes, that's what i'm aiming for. However, as you remember, some of these changes produce unwanted side-effects I remember you said there were side effects, but

Re: Doc: NR Pitches.itely - added 2 new snippets (issue 110240044 by pkx1...@gmail.com)

On Mon, 30 Jun 2014 14:40:17 -0700, pkx1...@gmail.com wrote: Is it worth making the Notation Reference longer? What do you suggest? Depending on your opinion about the NR length, maybe leaving these examples in lsr only.

Re: Issue 3254: align unassociated lyrics using NoteColumn extent. (issue 108110044 by janek.lilyp...@gmail.com)

On Thu, 26 Jun 2014 01:01:11 -0700, janek.lilyp...@gmail.com wrote: pushed as c73b41b3e7be6d7280c6336cf03610cd7aed3000 - thanks for review! I'll post a follow-up patch later today. https://codereview.appspot.com/108110044/ It seems that after this patch, the old well-tested Devnull context

Re: line-breaking bug in 2.19

On Sun, 16 Feb 2014 23:50:06 -0800, Keith OHara k-ohara5...@oco.net wrote: On Sun, 16 Feb 2014 17:33:16 -0800, Keith OHara k-ohara5...@oco.net wrote: On Sun, 16 Feb 2014 13:58:01 -0800, Janek Warchoł janek.lilyp...@gmail.com wrote: 2014-01-17 17:55 GMT+01:00 Keith OHara k-ohara5...@oco.net

Re: line-breaking bug in 2.19

On Mon, 17 Feb 2014 00:24:22 -0800, Keith OHara k-ohara5...@oco.net wrote: On Sun, 16 Feb 2014 23:50:06 -0800, Keith OHara k-ohara5...@oco.net wrote: On Sun, 16 Feb 2014 17:33:16 -0800, Keith OHara k-ohara5...@oco.net wrote: On Sun, 16 Feb 2014 13:58:01 -0800, Janek Warchoł janek.lilyp

Re: line-breaking bug in 2.19

On Sun, 16 Feb 2014 13:58:01 -0800, Janek Warchoł janek.lilyp...@gmail.com wrote: 2014-01-17 17:55 GMT+01:00 Keith OHara k-ohara5...@oco.net: Starting with version 2.19.0, the bassoon part from http://www.mutopiaproject.org/cgibin/piece-info.cgi?id=1793 gets a short line. I have tried

Re: line-breaking bug in 2.19

On Sun, 16 Feb 2014 17:33:16 -0800, Keith OHara k-ohara5...@oco.net wrote: On Sun, 16 Feb 2014 13:58:01 -0800, Janek Warchoł janek.lilyp...@gmail.com wrote: 2014-01-17 17:55 GMT+01:00 Keith OHara k-ohara5...@oco.net: Starting with version 2.19.0, the bassoon part from http

Re: NR: Removed example for alternate repeats (issue 61170044)

On Mon, 10 Feb 2014 04:31:25 -0800, pkx1...@gmail.com wrote: These three examples are more useful as regression tests than as documentation. Are they robust enough? Do they exercise each property properly? They seem good enough, but it is not essential to add them to /input/regressoin/

Re: regtest for new glyphs necessary?

Marc Hohl marc at hohlart.de writes: I am about to upload a patch for these four glyphs within the next days, but I wanted to ask whether a new (or enhanced existing) regtest is necessary for this type of enhancement? They could conceivably be broken accidentally, by a mistaken change to our

Re: note-spacing: stretch somewhat uniformly; issue 3304 (issue 36830045)

On Thu, 26 Dec 2013 22:47:41 -0800, Trevor Bača trevorb...@gmail.com wrote: I suppose it's important to note that all of my scores use proportional notation everywhere. So I imagine that most of the horizontal spacing improvements don't actually impact my scores. (The recent spacing changes

Re: Automatically unfold percent and tremolo repeats in MIDI output. (issue 769) (issue 40720060)

On Sun, 22 Dec 2013 11:45:49 -0800, Devon Schudy dsch...@gmail.com wrote: Unfolding during iteration doesn't extend well to voltas, because it changes their length. Even if the length is updated, the old length has already been used by the parent iterator to compute moments for subsequent

Re: Automatically unfold percent and tremolo repeats in MIDI output. (issue 769) (issue 40720060)

On Sat, 21 Dec 2013 00:23:05 -0800, d...@gnu.org wrote: Keith comments: The other limitation of \unfoldRepeats is that it fails to see repeats in parallel expressions {\repeat volta 2 s1 } {c2 d2} while this approach using the iterators will see them and be able to expand them when

Re: anyone notice speed of 2.17.95 on Windows ?

On Sat, 14 Dec 2013 23:57:20 -0800, Mike Solomon m...@mikesolomon.org wrote: On Dec 15, 2013, at 12:58 AM, Keith OHara k-ohara5...@oco.net wrote: The most transparent reorganization might be to have all properties that can hold data use the usual callback mechanism: where the callback

Re: anyone notice speed of 2.17.95 on Windows ?

On Sat, 14 Dec 2013 00:49:43 -0800, Mike Solomon m...@mikesolomon.org wrote: On Dec 14, 2013, at 9:35 AM, David Kastrup d...@gnu.org wrote: Most of the contortions seem focused about when or when not and how to pass begin/end columns. It would seem to make sense to turn them into dynamic

Re: anyone notice speed of 2.17.95 on Windows ?

On Sat, 14 Dec 2013 14:58:52 -0800, Keith OHara k-ohara5...@oco.net wrote: Several considered line-breaks might have the same 'start' and 'end' around the hairpin, but the upper-level axis-group structure caches results for each start/end pair so just one skyline is generated for each

Re: anyone notice speed of 2.17.95 on Windows ?

On Thu, 12 Dec 2013 23:07:19 -0800, Keith OHara k-ohara5...@oco.net wrote: I am happy to say that I was wrong here. Open_type_font:: and Pango_font::name_to_index() each call FT_Get_Name_Index(). Inserting print statements to trace those calls I find that FT_Get_Name_Index is called: 7

Re: anyone notice speed of 2.17.95 on Windows ?

On Fri, 13 Dec 2013 01:38:23 -0800, Werner LEMBERG w...@gnu.org wrote: I will check whether I can improve that in FreeType. Is there a reason this would have such a significantly different impact in Windows? No. It's exactly the same code on both platforms. However, for lilypond, even a

Re: anyone notice speed of 2.17.95 on Windows ?

On Dec 13, 2013, at 9:07 AM, Keith OHara k-ohara5...@oco.net wrote: Open_type_font:: and Pango_font::name_to_index() each call FT_Get_Name_Index(). Inserting print statements to trace those calls I find that FT_Get_Name_Index is called: 7 times for each character in a Tempo The layers

Re: anyone notice speed of 2.17.95 on Windows ?

On Fri, 13 Dec 2013 23:05:49 -0800, David Kastrup d...@gnu.org wrote: That does not make sense. If you want call-once behavior, you can just use a callback. At the moment, the decision on whether to preserve the callback pointer in the grob property, or fill the property with the returned

Re: anyone notice speed of 2.17.95 on Windows ?

On Tue, 10 Dec 2013 22:44:18 -0800, Keith OHara k-ohara5...@oco.net wrote: After a brief look at the code, it looked like for each glyph being laid out there is one extra call of Open_type_font:: or Pango_font::name_to_index(), which is the function called by find_by_name() that caused

Re: anyone notice speed of 2.17.95 on Windows ?

On Mon, 25 Nov 2013 00:10:08 -0800, David Kastrup d...@gnu.org wrote: Keith OHara k-ohara5...@oco.net writes: I timed one big score, Movement 1 of http://www.mutopiaproject.org/cgibin/piece-info.cgi?id=1793 2.16.2 2.17.95 WinXP 2m 30s 5m 10s Fedora 1m 50s 1m 50s Have you

Re: anyone notice speed of 2.17.95 on Windows ?

On Tue, 10 Dec 2013 01:59:09 -0800, Phil Holmes m...@philholmes.net wrote: When I benchmarked with and without skylines, I found there was only a noticeable difference with a lot of markup or similar: normal music had almost no effect. As a result, I concluded with skylining was the correct

Re: anyone notice speed of 2.17.95 on Windows ?

On Tue, 10 Dec 2013 09:57:20 -0800, Keith OHara k-ohara5...@oco.net wrote: On Tue, 10 Dec 2013 01:59:09 -0800, Phil Holmes m...@philholmes.net wrote: When I benchmarked with and without skylines, I found there was only a noticeable difference with a lot of markup or similar: normal music had

Re: anyone notice speed of 2.17.95 on Windows ?

On Tue, 10 Dec 2013 17:22:19 -0800, David Kastrup d...@gnu.org wrote: Keith OHara k-ohara5...@oco.net writes: The last time we had a doubling of time required on Windows relative to Linux, issue 1926, it was repeated calls to find_by_name() that go through Pango to the font server. Here

Re: Reduce offsets of \super and \sub (issue 35320043)

On Sat, 30 Nov 2013 23:56:09 -0800, janek.lilyp...@gmail.com wrote: I have one request: this patch makes the situation better, and even if the baseline-skip approach is wrong, it was already used that way so it's not making things worse. I suggest to push this patch, and then work on making

Re: Reduce offsets of \super and \sub (issue 35320043)

On Sat, 30 Nov 2013 21:21:49 -0800, lemzw...@googlemail.com wrote: Would it be possible to directly compute the height of the current font's `x' glyph as a basis value for raising and lowering? https://codereview.appspot.com/35320043/ I tried to do that for centering dynamics on their

Re: script-column: earlier scripts support later scripts; issue 3683 (issue 35010043)

On Fri, 29 Nov 2013 00:20:11 -0800, d...@gnu.org wrote: https://codereview.appspot.com/35010043/diff/1/lily/script-column.cc#newcode156 lily/script-column.cc:156: use all the scripts so far as support for the current grob A question of understanding: supports are not transitive automatically

  1   2   3   4   5   6   >