Re: LSR updates: was: polychords: a working solution

[David Nalesnik]

David,

On Tue, Feb 21, 2012 at 10:37 AM, David Kastrup <address@hidden> wrote:

David Kastrup <address@hidden> writes:

> David Nalesnik <address@hidden> writes:
>
>> David,
>>
>> On Tue, Feb 21, 2012 at 9:24 AM, David Kastrup <address@hidden> wrote:
>>
>>     Uh, convertrules.py converts interval-translate to coord-translate
>>     so
>>     where is the actual problem?
>>
>>
>>
>> Certainly coord-translate is the natural fix (thank you!), but when I
>> run convert-ly and the snippet is updated to 2.14.0, this replacement
>> isn't made.  I looked and didn't find a rule for it in convertrules.py
>> for 2.14.2.
>
> fc335d9e python/convertrules.py (Neil Puttock 2011-08-18 00:01:32
> +0100 2953) @rule ((2, 13, 27),
> fc335d9e python/convertrules.py (Neil Puttock 2011-08-18 00:01:32
> +0100 2954) ("interval-translate -> coord-translate"))
> fc335d9e python/convertrules.py (Neil Puttock 2011-08-18 00:01:32
> +0100 2955) def conv (str):
> fc335d9e python/convertrules.py (Neil Puttock 2011-08-18 00:01:32
> +0100 2956) str = re.sub ('interval-translate', 'coord-translate',
> str)
> fc335d9e python/convertrules.py (Neil Puttock 2011-08-18 00:01:32
> +0100 2957) return str
> fc335d9e python/convertrules.py (Neil Puttock 2011-08-18 00:01:32
> +0100 2958)
>
> Apparently added as a rule in 2.15.9.  Is there a reason you are not
> using the _current_ convert-ly to do the conversion?  It is not to be
> expected that old versions do a better job than current versions at
> converting old versions.

I mean, we are talking about _improving_ the convert-ly rules such that
they will presumably work for upgrading 2.12.

Of course, that is utterly pointless if you are using the 2.14.2
convert-ly for conversion.  If you are not going to use the newest
convert-ly, any fixes that are made to convert-ly will be totally
useless.

So please check with the _current_ convert-ly.  It has an option

  -t, --to=VERSION     convert to VERSION [default: 2.15.31]

for telling it at which version to stop.  You don't need to let it run
all through to 2.15.31.

And conversions that fail with _that_ convert-ly are worth further
investigation.

--
David Kastrup

Thank you very much for this information--I didn't know about this option.

-David