-
-
Notifications
You must be signed in to change notification settings - Fork 565
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update ECL to 10.4.1 and Maxima to 5.22.1 - currently the latest releases. #10187
Comments
comment:2
The doctests show one maxima regression:
This simplified to 0 in a previous version. But at least its not wrong ;-) The attached patch fixes all doctests except for a timeout in maxima. That one is a different issue with maxima, which forgets to print the
|
comment:3
FYI, here's the failures on OpenSolaris 06/2009 with those updated versions of Maxima and ECL.
I've not applied any patches at this point. I'll apply patches and rerun later. |
comment:4
Replying to @sagetrac-drkirkby:
I'd be interested what happened if you manually removed the tests in that file which test the tab-completion for Maxima. I get a problem on Mac OS X Tiger with that file for exactly that reason. We haven't filed a ticket because nobody knows how to fix it, and tab-completion does work on that platform, just not the testing. |
comment:5
The TIMED OUT! error is the missing I've filed a bug upstream here: https://sourceforge.net/tracker/?func=detail&aid=3098375&group_id=4933&atid=104933 |
comment:6
Replying to @vbraun:
Okay, thanks - so that's unrelated to the Tiger timeout from earlier Maximas. |
comment:7
I've modified This requires that `$SAGE_LOCAL/bin/sage-maxima.lisp sets
I've made a updated sage_scripts spkg here: To test this ticket, you need all three spkgs and both patches. |
Upstream: Workaround found; Bug reported upstream. |
comment:10
One thing we don't don, which I think would be sensible, is to include the test suite for ECL. That adds 1.7 MB. From the README.ECL file:
Would it not be sensible to add these in? Then we could execute the tests with a spkg-check file, like we do other code which has self-tests. Dave |
comment:11
Replying to @vbraun:
Just a very minor note:
but of course you changed it to get rid of the extra two lines from our constantly added packages :) so it's now five lines. Might as well change that too so future people are confused. |
comment:13
I looked trough doctest patch and compared it with mine from #8731, I have few comments. I think that doctest fix for sage/calculus/functional.py should be modified - it was documentation of the use of assume, but now - when maxima can calculate this integral without assumption, isn't that same as applying assumption to result? And the integral is checked in other doctest now. Maybe there is better example that still needs the assume to get result? Also the doctest in sage/misc/functional.py was here to demonstrate numerical approximation of integral that cannot be evaluated iirc, and now when it can be evaluated, I think it should be changed to something that cannot be - like Jason did in patch in #8731. Finally, there is typo in doctest to sage/symbolic/expression.pyx - it should say that this doctest is here to check that #7334 not #7344 is fixed (#7344 point to libjpeg issue so it cannot be it) - and it should stay in here unmodified. Also - the log thing - it's not exactly regression, it's feature. This is because of Maxima ticket 947808 - http://sourceforge.net/tracker/?func=detail&aid=947808&group_id=4933&atid=104933 - they now try to keep the expression as factored as they can without using factor. This results in observed behaviour. This doctest I think cannot be changed as it demonstrated that ticket is solved, but we should apply "x = x.simplify_rational()" after "x = x.simplify_log('one')" in definition of simplify_full - that way full simplify would result in same results as before I think, and #7334 would be still fixed. Now, this doctest shows nothing. |
comment:14
There is a problem with the ECL package here. The history of all recent changes has been lost. The recent changes to the package have been:
but instead SPKG.txt shows
This error has occurred since the package is based on one that I created several months ago, which never got merged, because their were conflicts with that and the Maxima package at the time. The current version of ECL in Sage should have been used as a starting point - not an old one that was never merged. I'll sort the above problem out, and add the ECL test code today. Dave |
comment:15
The patch fix_easy_doctest doesn't apply cleanly on either 4.6.rc0 or 4.6.1.alpha0. |
comment:16
Replying to @vbraun:
Could you please attach a patch to the scripts repo, too (rather than a link to a complete new scripts spkg)? Also, attaching diffs (or Mercurial patches) of the spkgs makes reviewing easier. |
comment:17
Replying to @nexttime:
P.S.: Of course I could do that, too, but then you wouldn't be able to update it in case you later modify the patch. |
Attachment: trac_10187_general_display_prefix_workaround.patch.gz Updated patch |
comment:18
I've incorporated your suggestions and updated the patches. Both are (and were) against 4.6.rc0 and apply cleanly. Maybe you had the wrong order? It should be
I'll attach patches for the |
comment:19
Wrong order indeed! I thought the two patches were orthogonal. |
comment:20
I'm going to have to ask on the ECL list how to run the test suite - I can't work out where ones supposed to copy the source. These changes are not going to make it into 4.6. The milestone is 4.6.1, and it will easily be resolved by then. Dave |
comment:21
Replying to @sagetrac-drkirkby:
I've now got the information on the ANSI test suite from the ECL developer, though I gather the copy on the ECL site is rather out of date. Also, the ECL developer has fixed the bug that causes #9840, so I'll patch that too, so #9840 can hopefully be closed at the same time. I'll try to get this sorted out within the next few days. Dave |
comment:22
I have put an updated ECL file at http://boxen.math.washington.edu/home/kirkby/patches/ecl-10.4.1.spkg This has only been checked on OpenSolaris - I don't have access to a Fedora 14 machine, so can't verify if it actually fixes the issues reported at #10185 I should make a few comments about this:
After
I get one reject out of six when adding
to sage 4.6.1.alpha0, so I think that patch needs updating. The contents of the reject are:
But as I say, all doctests passed for me, despite one patch was not fully applied!!
I'm leaving as needs work, as clearly the fact one patch does not apply cleanly is a problem. It also needs testing on more than one system, but I don't have access to the Fedora 14 system where this was a particular problem. Dave |
This comment has been minimized.
This comment has been minimized.
Changed author from Volker Braun to Volker Braun, David Kirkby |
comment:139
Yes, Another |
comment:140
Am I right in assuming all these patches are against sage-4.6.1.alpha1 ? I just built sage-4.6.1.alpha2, but some of these patches are applied, and some are not. So it looks like I'm going to have to build sage-4.6.1.alpha1. I expect it will take me a couple of hours to build it, patch it, then doctest it Dave |
comment:141
Replying to @sagetrac-drkirkby:
Shouldn't be necessary. I've previously tested the spkgs and patches successfully ( On the 32-bit machine, I so far again got 47 doctest errors with the same though. |
comment:142
Hmmm, maybe a Mercurial issue? Patches apply apparently clean on vanilla alpha2 as well (with an old Mercurial version), but I also get doctest errors on Ubuntu 9.04 x86_64. |
comment:143
Replying to @nexttime:
Same with Sage's Mercurial version (1.6.x), i.e. patches apply cleanly. Now at 199+ doctest errors on the 32-bit machine... :( |
comment:144
Ubuntu 9.04 x86_64 (gcc 4.3.3), vanilla Sage 4.6.1.alpha2 with #10187:
Currently no idea what happens there... |
comment:145
Replying to @nexttime:
Now I get (Ubuntu 9.04 x86, gcc 4.3.3):
which at first glance doesn't look much different; 525 doctest errors. |
comment:146
(Note that on all systems vanilla Sage 4.6.1.alpha2 passes all tests.) |
comment:147
If you would tell us what the error is then that would be much more useful than a list of failed tests. All of these work fine on Fedora 14 with sage-4.6.1.alpha2 and the new maxima/ecl. My first guess would be that you forgot to apply one of the patches (been there, done that), can you double-check? |
comment:148
Well, I didn't attach the logs since they're huge. I've reverted all changes and started from scratch, and now all tests pass on the Ubuntu 9.04 x86_64 machine. I have absolutely no idea what went wrong in the first two attempts; in the succeeding one I used Sage's I did apply all patches in the right order in all cases, so something else must be wrong. Never experienced similar before... |
comment:149
The patches on this ticket are not so easy to apply as usual, as part of them is already in Sage 4.6.1.alpha2. What we don't have on this ticket are changes relative to 4.6.1.alpha2. I've built sage 4.6.1.alpha1 with the patches applied and the new .spkg files. This is on a Sun Ultra 27, 3.33 GHz Xeon processor running OpenSolaris 06/2009.
I wonder if Leif's problems had anything to the patch process going wrong? Dave |
comment:150
Replying to @sagetrac-drkirkby:
Really? If that's true, it was unintentional. Which patches are in sage-4.6.1.alpha2? |
comment:151
Replying to @jdemeyer:
I don't think so. I've applied all patches multiple times with One problem I ran into seems to be that when running Another odd thing I discovered is that at least on Ubuntu 9.04, |
comment:152
Replying to @nexttime:
... which means one has to apply patches to the Sage library after all spkgs have been installed that way. Also, there seems to be a missing dependency for |
comment:153
Replying to @jdemeyer:
I guess I must have been mistaken - perhaps my alpha2 was not "clean", though I thought I'd extracted a fresh tarball. Anyway, several others have got all passes, and I got all passes if I apply this to sage-4.6.1.alpha1. I would say I'd try again with sage-4.6.1.alpha2, but it is late here, and I'm not going to start a build now, as you wont get the results until I wake in the morning. I'm pretty convinced this is OK. To my knowledge, the only thing was some minor changes Leif wanted to the ECL package, and those have been addressed. Leif seemed happy, so I'm setting this to positive. |
comment:154
Replying to @nexttime:
I'm not surprised by that, as When building gcc, the Dave Dave |
comment:155
I'm just changing the ticket title, to be more explicit. |
This comment has been minimized.
This comment has been minimized.
comment:156
Replying to @sagetrac-drkirkby:
Yes, but I would have expected the Debian or Ubuntu developers to set the Not many people like setting |
comment:157
Finally passed all tests (
Sorry for the noise. |
Merged: sage-4.6.1.alpha3 |
comment:159
I just wanted to say "Bravo!" to everyone that has diligently worked on this. Thanks for your work! |
Note: See #10434 for a follow-up ticket.
Please update ECL and Maxima to the newest upstream release. Sage packages are here:
It's unsafe to build either ECL or Maxima in parallel, so we must not do this. Updated versions of ECL and Maxima can be found here:
http://boxen.math.washington.edu/home/kirkby/patches/ecl-10.4.1.spkg
http://www.stp.dias.ie/~vbraun/Sage/spkg/maxima-5.22.1.spkg
Note that you cannot upgrade one without the other; Both ECL and Maxima need to be upgraded simultaneously or build will fail.
Relevant tickets for ECL:
Remove extra baggage from the ECL spkg #9493: Remove extra baggage from ECL 10.2.1.p1 (again). The package has been cleaned up, but I feel removing gmp sources is too dangerous.
ECL in Sage will not build on Fedora 14, which will be released on 2nd November 2010 #10185: the old ecl-10.2.1 does not build on Fedora 14: Hopefully fixed with the updated ECL from this ticket.
Relevant tickets for maxima:
maxima package fails to install ECL library #8645: ECL library "maxima.fasb" is built at new location. The patches from this bug have been incorporated into the maxima spkg, and maxima package fails to install ECL library #8645 can be closed after this ticket.
update/upgrade maxima to latest upstream (5.21.1) #8731: update/upgrade Maxima to latest upstream (5.21.1): Latest upstream is nowadays 5.22.1.
sum(1/(1+k^2), k, -oo, oo) returns 0 #8582:
sum(1/(1+k^2), k, -oo, oo)
returns 0: The new Maxima seems to be able to evaluate this correctly, at leasts it returns some expression with digamma functions.The updated Maxima code seems to be more careful about signs which leads to doctest errors. Moreover, error reporting was changed. Therefore you need to apply the following patches to the Sage library (in this order):
trac_10187_fix_easy_doctests.patch
trac_10187_general_display_prefix_workaround.patch
trac_10187_mark_some_doctests_random_until_9880_gets_merged.patch
Finally, you need to patch
$SAGE_ROOT/local/bin/sage-maxima.lisp
withtrac_10187_sage-maxima_lisp.patch
Upstream: Workaround found; Bug reported upstream.
CC: @williamstein @jhpalmieri @qed777 @nexttime @jaapspies @jasongrout @kcrisman @kiwifb @jpflori
Component: packages: standard
Author: Volker Braun, David Kirkby
Reviewer: Karl-Dieter Crisman, David Kirkby, Volker Braun, Leif Leonhardy
Merged: sage-4.6.1.alpha3
Issue created by migration from https://trac.sagemath.org/ticket/10187
The text was updated successfully, but these errors were encountered: