#17631: Auto-generate gen.pyx -- part 1
-------------------------+-------------------------------------------------
Reporter: | Owner:
jdemeyer | Status: needs_info
Type: | Milestone: sage-6.5
enhancement | Resolution:
Priority: major | Merged in:
Component: | Reviewers: Vincent Delecroix
interfaces | Work issues:
Keywords: | Commit:
Authors: | a5a6b25ad3c322c03cec85b6c85da4fcb5508cdf
Jeroen Demeyer | Stopgaps:
Report Upstream: N/A |
Branch: |
u/jdemeyer/17631 |
Dependencies: |
#16997 |
-------------------------+-------------------------------------------------
Comment (by jdemeyer):
Replying to [comment:22 vdelecroix]:
> > > Why did you keep so much methods in the `gen` class in `gen.pyx`
(such as `List`, `Mat`, `sign`, `vecmin`, `vecmax`, etc)?
> > In any case, I didn't want to remove any doctests. I also didn't check
very carefully which methods could be removed.
>
> Right. Doctest are indeed important! What is your long term goal with
those methods?
In the end, I think it's probably best to store all extra documentation
and doctests in a separate file (I haven't thought about the exact
mechanism). Then that extra documentation will be put together with the
PARI documentation in the auto-generated file.
> Just for precision: is `pari_catch_sig_on` just an equivalent to
`sig_on`?
Yes (although you're recommended to use `pari_catch_sig_on` in PARI code
in case we want to change this).
> I really do not understand why there is so much `pari_catch_sig_on`
without any `pari_catch_sig_off`!
Because the `new_gen()` method runs `pari_catch_sig_off()`.
--
Ticket URL: <http://trac.sagemath.org/ticket/17631#comment:23>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
