<br><br><div class="gmail_quote">On Thu, Aug 16, 2012 at 12:46 PM, Fabrice Silva <span dir="ltr">&lt;<a href="mailto:silva@lma.cnrs-mrs.fr" target="_blank">silva@lma.cnrs-mrs.fr</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Le mercredi 15 août 2012 à 20:54 +0200, Ralf Gommers a écrit :<br>
<div class="im">&gt; I was mixing it up a bit, but yes: the _odepack extension and the C<br>
&gt; source for it. Not necessary to do that at once I guess, but wrapping<br>
&gt; the same function twice is once too many.<br>
&gt;<br>
&gt; And forgot in my first email: nice PR, looks good to me.<br>
<br>
</div>OK then, you can found two commits :<br>
<br>
the first one removes the _odepack extension (and the relative<br>
multipack.h, __odepack.h and _odepackmodule.c), replacing it by Python<br>
counterparts in the odeint function itself.<br>
<a href="https://github.com/FabricioS/scipy/commit/02e8a4856f29f4ad438fef2c86a41b266d6a9e6c" target="_blank">https://github.com/FabricioS/scipy/commit/02e8a4856f29f4ad438fef2c86a41b266d6a9e6c</a><br>
<br></blockquote><div>Thanks.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
the second one suggests reverting callback arguments convention:<br>
        ydot = f(y,t,..)<br>
to ode&#39;s one:<br>
        ydot = f(t,y,..)<br>
This ones would raise backward compatibility issues but align ordering<br>
to the convention defined in the LLNL when designing the ODEPACK.<br>
<a href="https://github.com/FabricioS/scipy/commit/f867f2b8133d3f6ea47d449bd760a77a7c90394e" target="_blank">https://github.com/FabricioS/scipy/commit/f867f2b8133d3f6ea47d449bd760a77a7c90394e</a><br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div>This is probably not worth the cost for existing users imho. It is a backwards compatibility break that doesn&#39;t really add anything except for some consistency (right?).<br>
<br>Ralf<br><br></div></div>