update pynac to 0.2.1

[burcin]

There is a new pynac version available with patches for #9394, #9834, #9878, #9879, #9881, #9900.

http://sage.math.washington.edu/home/burcin/pynac/pynac-0.2.1.spkg

Patches from the tickets mentioned above need to be merged to test this.

This package depends on the patches available at #9394, #9878, #9879, #9881, #9900. In turn, the first one of these (#9394) depends on #8568 because of changes in sage/symbolic/random_tests.py. The following patches should be applied before running tests:

• attachment: trac_8568-diff_conversion.take2.patch:ticket:8568
• attachment: trac_8568-erf-deriv.patch:ticket:8568
• attachment: trac_8568-fix_doctests.patch:ticket:8568
• attachment: trac_9394-leading_minus.patch:ticket:9394
• attachment: trac_9878-zeta_infinity.patch:ticket:9878
• attachment: trac_9879-hold.patch:ticket:9879
• attachment: trac_9881-csgn.patch:ticket:9881
• attachment: trac_9900_conjugate_doctests.patch:ticket:9900

CC: @qed777

Component: symbolics

Keywords: pynac

Author: Burcin Erocal

Reviewer: Karl-Dieter Crisman

Merged: sage-4.6.alpha3

Issue created by migration from https://trac.sagemath.org/ticket/9901

[burcin]

Description changed:

--- 
+++ 
@@ -2,5 +2,16 @@
 
 http://sage.math.washington.edu/home/burcin/pynac/pynac-0.2.1.spkg
 
-Patches from the tickets mentioned above need to be merged to test this. I'll post detailed instructions after I upload the patches.
+Patches from the tickets mentioned above need to be merged to test this. 
 
+This package depends on the patches available at #9394, #9878, #9879, #9881, #9900. In turn, the first one of these (#9394) depends on #8568 because of changes in `sage/symbolic/random_tests.py`. The following patches should be applied before running tests:
+
+* [attachment: trac_8568-diff_conversion.take2.patch:ticket:8568](https://github.com/sagemath/sage/files/ticket9901/b8e6d0c311a619dbd062faaf89bfe343.gz)
+* [attachment: trac_8568-erf-deriv.patch:ticket:8568](https://github.com/sagemath/sage/files/ticket9901/f8e94fc15f353075ab932331b122cc3c.gz)
+* [attachment: trac_8568-fix_doctests.patch:ticket:8568](https://github.com/sagemath/sage/files/ticket9901/74d2bee7d9f54c5c72e5e0b2943a513c.gz)
+* [attachment: trac_9394-leading_minus.patch:ticket:9394](https://github.com/sagemath/sage/files/ticket9901/a70768099479e88012e3a92963a885be.gz)
+* [attachment: trac_9878-zeta_infinity.patch:ticket:9878](https://github.com/sagemath/sage/files/ticket9901/fd70ae63b1f82c47b1d6c823608395ec.gz)
+* [attachment: trac_9879-hold.patch:ticket:9879](https://github.com/sagemath/sage/files/ticket9901/c82dabdff86ad2440729edf062afa894.gz)
+* [attachment: trac_9881-csgn.patch:ticket:9881](https://github.com/sagemath/sage/files/ticket9901/445550ac592d6306db365326b8340e9b.gz)
+* [attachment: trac_9900_conjugate_doctests.patch:ticket:9900](https://github.com/sagemath/sage/files/ticket9901/d5fe6cd66fe4b381b01e2a6f94da5e9d.gz)
+

[kcrisman]

comment:2

I'm going to try (!) to review all this soon. Just a point: the Pynac website has no mention of this update.

[kcrisman]

comment:3

Most of the changesets are pretty obviously related to the tickets in question, and I'll put comments there. Here are a few things which I will put here for archiving which came up in discussion, because a few changesets in Pynac are nonobvious.

> As far as Pynac is concerned:
>
> "Do not generate function.{h,cpp} automatically."  How do I check
> this?  What is the reasoning for this?  How do you test this in the
> Sage library (say, for 13 arguments), as it probably should be?  Why
> do you keep it for (say) 3, but not 4?

You can't check this in the Sage library. The auto generated calls were
something we inherited from GiNaC as a convenience for programming in
C++. Since we use Python, which uses a function call mechanism very
similar to the vector arguments case GiNaC supports, there is no reason
to keep the auto generated functions.

I kept the calls up to 3, since functions implemented in GiNaC use at
least up to 2. I don't know if there is anything which needs 3.

> Are some of these things items you would want to report upstream?    I
> am thinking of the conjugate handling, for instance, or the unsigned
> infinity.  Maybe they come from upstream?

> Changeset 172 - what is that all about?

It just deletes some files from the mercurial repository. They are
generated by autoconf. William just imported everything in the GiNaC
distribution tarball when he started working on pynac. This is just
cleaning up some of that cruft.

Incidentally, we also note that the conjugate handling change came from upstream.

[kcrisman]

Reviewer: Karl-Dieter Crisman

[kcrisman]

comment:4

Update for release manager: positive review for all but #9879 and #9900. #9879 awaits a review of the reviewer patch (just because it's somewhat long), and #9900 will be positive review once the reviewer creates a reviewer patch, hopefully sometime tomorrow.

Also, although the Pynac website doesn't mention these changes yet, the changesets ARE available here, so it is on the web and not just in Sage :)

[jpflori]

comment:6

I only run at the docsets and everything went fine.
I'll have a look at #9879 and #9900 later.

[kcrisman]

comment:7

Based on the now positively reviewed #9879 and #9900, this should have positive review. Hooray!