## #8903 closed defect (fixed)

# update pynac to 0.2.0

Reported by: | Burcin Erocal | Owned by: | tbd |
---|---|---|---|

Priority: | major | Milestone: | sage-4.4.3 |

Component: | packages: standard | Keywords: | pynac |

Cc: | Mike Hansen | Merged in: | sage-4.4.3.alpha1 |

Authors: | Mike Hansen, Burcin Erocal | Reviewers: | William Stein |

Report Upstream: | N/A | Work issues: | |

Branch: | Commit: | ||

Dependencies: | Stopgaps: |

### Description

A new pynac package with several critical fixes is available here:

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

It contains fixes for:

- #8542: function table for Cygwin
- #8651: binomial(n, 0) -> 1
- #8688: extra parenthesis when typesetting fractions
- #8775: auto evaluation of conjugates

Note that patches from the above tickets need to be applied to test this ticket. Without #8542, you'll get a segfault. The others fix doctests.

### Change History (15)

### comment:1 Changed 12 years ago by

Status: | new → needs_review |
---|

### comment:2 Changed 12 years ago by

### comment:3 Changed 12 years ago by

Status: | needs_review → needs_work |
---|

The last patch for conjugates and power simplification included in this pynac version breaks doctests in `sage/rings/qqbar.py`

and a bunch of other places.

I suppose the fix will involve looking into the `power_helper`

in detail, hopefully fixing #8959 on the way. I won't have time for this at least for a week though.

If anybody is interested in working on the cygwin port, I can prepare a package which includes only the patches relevant for that in the mean time.

### comment:4 Changed 12 years ago by

I've put a new spkg up at http://sage.math.washington.edu/home/burcin/pynac/pynac-0.2.0.spkg . This just has a simple fix for #9307.

### comment:7 Changed 12 years ago by

I've made a new spkg at http://sage.math.washington.edu/home/burcin/pynac/pynac-0.2.0.spkg which just backs out the commit which adds "auto evaluation of conjugates". All tests in qqbar pass. We can add the auto evaluation of conjugates in 0.2.1.

### comment:8 Changed 12 years ago by

Resolution: | → fixed |
---|---|

Status: | needs_work → closed |

### comment:9 Changed 12 years ago by

Merged in: | → 4.4.3.alpha1 |
---|---|

Reviewers: | → wstein |

### comment:10 Changed 12 years ago by

Merged in: | 4.4.3.alpha1 → sage-4.4.3.alpha1 |
---|---|

Reviewers: | wstein → William Stein |

### comment:11 Changed 12 years ago by

Despite Mike's comments on #9037, this does not resolve the issue there, as it still has:

if [ `uname` = "Darwin" -a "$SAGE64" = "yes" ]; then echo "64 bit MacIntel" CXXFLAGS="-m64 -O2 -g"; export CXXFLAGS LDFLAGS="-m64"; export LDFLAGS fi

I'll have to create a new package based on this one and apply the fix again.

Dave

### comment:12 follow-up: 13 Changed 12 years ago by

Hi Dave,

Which package are you looking at. Unfortunately, there are several packages named pynac-0.2.0.spkg floating around. My version at

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

doesn't have the said fix, but Mike's version at

http://sage.math.washington.edu/home/mhansen/pynac-0.2.0.spkg

might do. I believe the version included in the release is the latter.

### comment:13 Changed 12 years ago by

Replying to burcin:

Hi Dave,

Which package are you looking at. Unfortunately, there are several packages named pynac-0.2.0.spkg floating around. My version at

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

doesn't have the said fix, but Mike's version at

http://sage.math.washington.edu/home/mhansen/pynac-0.2.0.spkg

might do. I believe the version included in the release is the latter.

I often wish there was a central repository, as having different versions and constant rebasing does get a bit annoying.

Mike's version at http://sage.math.washington.edu/home/mhansen/pynac-0.2.0.spkg does have the fix, but the only version mentioned on this trac ticket was yours at http://sage.math.washington.edu/home/burcin/pynac/pynac-0.2.0.spkg which does not have the fix. The comment from William was

"I merged in http://sage.math.washington.edu/home/burcin/pynac/pynac-0.2.0.spkg"

so I can only assume your version, and not Mikes is merged.

Actually, it appears changing

if [ `uname` = "Darwin" -a "$SAGE64" = "yes" ]; then

to

if [ "$SAGE64" = "yes" ]; then

(as in Mike's version), does not fully solve the 64-bit build issue on OpenSolaris. However, it is certainly a desirable change, so if you make any more changes to pynac, can you change that one line.

In the mean time, I'll work on trying to resolve why that is not a complete fix for the 64-bit OpenSolaris issue, but it is certainly a necessary change.

Dave

### comment:14 follow-up: 15 Changed 12 years ago by

I'll make sure that mine is the one in 4.4.3.alpha1.

### comment:15 Changed 12 years ago by

Replying to mhansen:

I'll make sure that mine is the one in 4.4.3.alpha1.

The ticket says Burchin's package has already been merged in sage-4.4.3.alpha1 - whether it is possible to reverse that easily I don't know.

Dave

**Note:**See TracTickets for help on using tickets.

This package depends on the Python package at #8907.