Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#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:

Status badges

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 Burcin Erocal

Status: newneeds_review

comment:2 Changed 12 years ago by Burcin Erocal

This package depends on the Python package at #8907.

comment:3 Changed 12 years ago by Burcin Erocal

Status: needs_reviewneeds_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 Mike Hansen

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:5 Changed 12 years ago by Mike Hansen

err, #8907

comment:6 Changed 12 years ago by Mike Hansen

Third time is the charm: #9037

comment:7 Changed 12 years ago by Mike Hansen

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 William Stein

Resolution: fixed
Status: needs_workclosed

comment:9 Changed 12 years ago by William Stein

Merged in: 4.4.3.alpha1
Reviewers: wstein

comment:10 Changed 12 years ago by Minh Van Nguyen

Merged in: 4.4.3.alpha1sage-4.4.3.alpha1
Reviewers: wsteinWilliam Stein

comment:11 Changed 12 years ago by David Kirkby

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 Changed 12 years ago by Burcin Erocal

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 in reply to:  12 Changed 12 years ago by David Kirkby

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 Changed 12 years ago by Mike Hansen

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

comment:15 in reply to:  14 Changed 12 years ago by David Kirkby

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.