#10036 closed enhancement (fixed)
SageNB 0.8.6
Reported by: | mpatel | Owned by: | jason, was |
---|---|---|---|
Priority: | major | Milestone: | sage-4.6 |
Component: | notebook | Keywords: | |
Cc: | acleone, jason, kcrisman, timdumol | Merged in: | |
Authors: | Mitesh Patel | Reviewers: | Jason Grout |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Merged in SageNB 0.8.3: #9000, #9148, #9327, #9822.
Merged in SageNB 0.8.4: #9920.
Merged in SageNB 0.8.5: #9828.
Merged in SageNB 0.8.6: #9920 (devel/
fix).
Release manager
Please merge
- http://sage.math.washington.edu/home/mpatel/trac/10036/sagenb-0.8.6.spkg
- sage repository patch at #9920: http://trac.sagemath.org/sage_trac/raw-attachment/ticket/9920/trac_9920-sagelib-hg.patch
- sage and scripts repository patches at #9828:
More specifically:
- install http://sage.math.washington.edu/home/mpatel/trac/10036/sagenb-0.8.6.spkg
- install the new Cython spkg at #9828: http://sage.math.washington.edu/home/robertwb/cython/cython-0.13.p1.spkg
- apply the Cython patch at #9828 to the Sage repository: http://trac.sagemath.org/sage_trac/raw-attachment/ticket/9828/trac_9828-cython_0_13.patch
- apply the Cython patch at #9828 to the Sage scripts repository: http://trac.sagemath.org/sage_trac/raw-attachment/ticket/9828/trac_9828-scripts_hudson.patch
- apply the hg_sage patch at #9920 to the Sage repository: http://trac.sagemath.org/sage_trac/raw-attachment/ticket/9920/trac_9920-sagelib-hg.patch
Then do sage -ba to refresh the Cython files.
Change History (20)
comment:1 Changed 8 years ago by
- Cc acleone jason kcrisman timdumol added
- Description modified (diff)
- Milestone set to sage-4.6
- Status changed from new to needs_review
comment:2 follow-up: ↓ 5 Changed 8 years ago by
It would be really great if #9920 were slipped into this as well.
What do we need to do to review this? We already reviewed the individual patches.
comment:3 Changed 8 years ago by
- Description modified (diff)
comment:4 Changed 8 years ago by
- Summary changed from SageNB 0.8.3 to SageNB 0.8.4
comment:5 in reply to: ↑ 2 Changed 8 years ago by
Replying to jason:
What do we need to do to review this? We already reviewed the individual patches.
I think you can just check that I didn't make a mistake in applying the right tickets, install the package, run the doctests.
comment:6 Changed 8 years ago by
A tentative SageNB 0.8.5 == 0.8.4 + #9828 is at
http://sage.math.washington.edu/home/mpatel/trac/10036/sagenb-0.8.5.spkg
comment:7 Changed 8 years ago by
- Description modified (diff)
comment:8 Changed 8 years ago by
Could someone please review the package?
comment:9 Changed 8 years ago by
- Summary changed from SageNB 0.8.4 to SageNB 0.8.5
comment:10 follow-up: ↓ 12 Changed 8 years ago by
There's an error building Sage from scratch. See #9920 for the problem.
comment:11 Changed 8 years ago by
- Description modified (diff)
- Summary changed from SageNB 0.8.5 to SageNB 0.8.6
comment:12 in reply to: ↑ 10 Changed 8 years ago by
comment:13 Changed 8 years ago by
- Description modified (diff)
I'm a little confused at what to do to test this. I've updated the description. Did I get everything?
comment:14 Changed 8 years ago by
- Description modified (diff)
comment:15 follow-up: ↓ 17 Changed 8 years ago by
In #10094 I'm switching on c++ support in cython-0.13, this changes (actually, fixes) one sage.misc.sageinspect.py
doctest. The analogous change needs to be made to sagenb.misc.sageinspect.py
when/if that patch is accepted. Its simple:
http://trac.sagemath.org/sage_trac/attachment/ticket/10094/trac_10094_fix_doctests.patch
comment:16 Changed 8 years ago by
- Status changed from needs_review to positive_review
Okay, I followed the steps in the description (using 4.6.alpha2), on a production server running behind an apache proxy. ptestlong passed all tests, and I was successfully able to log in, run worksheets, create accounts, etc. So positive review.
comment:17 in reply to: ↑ 15 Changed 8 years ago by
Replying to vbraun:
In #10094 I'm switching on c++ support in cython-0.13, this changes (actually, fixes) one
sage.misc.sageinspect.py
doctest. The analogous change needs to be made tosagenb.misc.sageinspect.py
when/if that patch is accepted. Its simple: http://trac.sagemath.org/sage_trac/attachment/ticket/10094/trac_10094_fix_doctests.patch
This change has already been made in one of the patches merged into 0.8.6, so nothing needs to be done.
comment:18 Changed 8 years ago by
- Reviewers set to Jason Grout
comment:19 Changed 8 years ago by
- Merged in set to sage-4.6.alpha3
- Resolution set to fixed
- Status changed from positive_review to closed
Thanks, Jason!
comment:20 Changed 8 years ago by
- Merged in sage-4.6.alpha3 deleted
I'm merging #10097's 0.8.7 package, which supersedes 0.8.6.
The long doctests pass for me with 4.6.alpha2 + SageNB 0.8.3 on sage.math.
My current local setup is limited, so I don't know about the Selenium tests. As far as I'm aware, we still need to update the SageNB Se tests for the latest released version of Selenium RC.