On Wed, 02 Sep 2009, Dirk Eddelbuettel wrote:
> 1.0.3-9 is no longer the current version as you know :)
oops... what happened? ;)) (just teasing, no follow up is due ;))

> On 2 September 2009 at 10:36, Yaroslav Halchenko wrote:
> | In [204]:numpy.__version__ is rpy.__version__
> | Out[204]:True
> Please explain what you intended to convey in the English language. I have no
> idea what this is about.  Hence, I will close this if there is no follow-up
> as there is nothing actionable for me here.

__version__ is used in python modules to contain/report version of the
corresponding module.  but in this case, due to 'from numpy import *' it
simply obtains __version__ from numpy and it gets exposed as
rpy.__version__.  Fix is really trivial:

$> diff -Naur rpy_orig.py rpy.py 
--- rpy_orig.py 2009-09-02 12:34:49.885577281 -0400
+++ rpy.py      2009-09-02 12:35:16.841053597 -0400
@@ -139,6 +139,7 @@
 
 # Version
 from rpy_version import rpy_version
+__version__ = rpy_version
 
 # Symbolic names for conversion modes
 TOP_CONVERSION = 4


so it could be applied without any foreseen sideeffects, and backward
compatible (so rpy_version is still available) ;)  And I consider this a
bug indeed (so imho it should not be silently closed without
action).  Minor but still a bug (disclaimer: exaggeration
follows).  Imagine versions of packages in debian all be reported as the
version of libc? how would you feel about it? 

> If you want rpy to change, discuss it in the rpy list and keep in mind that
> it is no longer all that actively maintained by Greg W -- so best to supply a
> tested patch.  Also note that the version dependency of rpy onto R has been a
> pain for years. I don;t expect any short-term fixes.

> rpy2 is actively maintained and may be a better choice.  Among other
> things, it copes better with R version changes.
thanks for the advice -- we are targeting switch to rpy2, but for
now we stick with rpy. Also, although it is somewhat a pain, it is, I
guess, more tested by now than the fresh glowing rpy2.

-- 
                                  .-.
=------------------------------   /v\  ----------------------------=
Keep in touch                    // \\     (yoh@|www.)onerussian.com
Yaroslav Halchenko              /(   )\               ICQ#: 60653192
                   Linux User    ^^-^^    [175555]





-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to