#3999: [with patch, needs work] Wrapper class to treat additive groups as
multiplicative goups
------------------------------+---------------------------------------------
Reporter: robertwb | Owner: somebody
Type: defect | Status: new
Priority: major | Milestone: sage-3.2.2
Component: basic arithmetic | Resolution:
Keywords: |
------------------------------+---------------------------------------------
Changes (by was):
* summary: [with patch, needs review] Wrapper class to treat additive
groups as multiplicative goups => [with patch,
needs work] Wrapper class to treat additive
groups as multiplicative goups
Comment:
REFEREE REPORT:
This bitrotted. I couldn't apply cleanly because of setup.py being
refactored. I fixed that by hand, but then this wouldn't compile:
{{{
[EMAIL PROTECTED]:~/build/sage-3.2.1.alpha1$ ./sage -br
----------------------------------------------------------
sage: Building and installing modified Sage library files.
Installing c_lib
scons: `install' is up to date.
Updating Cython code....
Building modified file sage/groups/multiplicative_wrapper.pyx.
Execute 1 commands (using 1 cpus)
python2.5 `which cython` --embed-positions --incref-local-binop
-I/home/was/build/sage-3.2.1.alpha1/devel/sage-main -o
sage/groups/multiplicative_wrapper.c
sage/groups/multiplicative_wrapper.pyx
Error converting Pyrex file to C:
------------------------------------------------------------
...
except:
if x == 1:
return self.one
raise
cpdef bint _has_coerce_map_from_(self, S) except -2:
^
------------------------------------------------------------
/home/was/build/sage-3.2.1.alpha1/devel/sage-
main/sage/groups/multiplicative_wrapper.pyx:113:10: C method
'_has_coerce_map_from_' not previously declared in definition part of
extension type
Error converting Pyrex file to C:
------------------------------------------------------------
...
cdef MultWrapperElement e =
<MultWrapperElement>PY_NEW(MultWrapperElement)
e._parent = self._parent
e._elt = elt
return e
cdef MonoidElement _mul_c_impl(self, MonoidElement right):
^
------------------------------------------------------------
/home/was/build/sage-3.2.1.alpha1/devel/sage-
main/sage/groups/multiplicative_wrapper.pyx:191:9: C method '_mul_c_impl'
not previously declared in definition part of extension type
Error converting Pyrex file to C:
------------------------------------------------------------
...
return e
cdef MonoidElement _mul_c_impl(self, MonoidElement right):
return
self._new(self._elt._add_c((<MultWrapperElement>right)._elt))
cdef MultiplicativeGroupElement _div_c_impl(self,
MultiplicativeGroupElement right):
^
------------------------------------------------------------
/home/was/build/sage-3.2.1.alpha1/devel/sage-
main/sage/groups/multiplicative_wrapper.pyx:194:9: C method '_div_c_impl'
not previously declared in definition part of extension type
Parallel build failed with status 256.
sage: There was an error installing modified sage library code.
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3999#comment:4>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---