Opened 6 years ago

Closed 6 years ago

#1278 closed enhancement (fixed)

make R an optional Sage package that builds on everything (in prep for making it non-optional)

Reported by: was Owned by: was
Priority: major Milestone: sage-2.9
Component: packages: standard Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

See emails on sage-devel for details. I'll post specific notes about what has to be done below.

Change History (2)

comment:1 Changed 6 years ago by was

But, it doesn't look like my patch to rpy was applied:

sage: rpy.r(2r)
2.0
sage: rpy.r(2)
---------------------------------------------------------------------------
<class 'rpy.RException'>                  Traceback (most recent call last)

/home/mike/src/rpy-1.0-RC3/<ipython console> in <module>()

/opt/sage/local/lib/python2.5/site-packages/rpy.py in __call__(self, s)
   297
   298     def __call__(self, s):
--> 299         return self.eval(self.parse(text=s))
   300
   301     def __help__(self, *arg, **kw):

<class 'rpy.RException'>: cannot convert from type 'sage.rings.integer.Integer'

There should be an rpymodule.c in the patches/ directory in
rpy-1.0-RC3 in my home directory on sage.math which makes it so that
rpy will check for a _rpy_ method to convert a Sage object to an rpy
one.

comment:2 Changed 6 years ago by mabshoff

  • Milestone changed from sage-wishlist to sage-2.9
  • Resolution set to fixed
  • Status changed from new to closed

This happened a while ago.

Note: See TracTickets for help on using tickets.