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
More doctests from the book "Calcul mathématique avec Sage" chapter "integration" #14318
Comments
Attachment: integration_doctest.sage.gz |
comment:2
What am I supposed to do with this file? Normally, additions to Sage are patches to the Sage library (in |
comment:5
I've made a proper patch. All tests pass with Sage 5.11. Ready for review. Paul |
Author: Laurent Fousse, Paul Zimmermann |
This comment has been minimized.
This comment has been minimized.
comment:6
ok, looks good to me |
Reviewer: Frédéric Chapoton |
This comment has been minimized.
This comment has been minimized.
comment:9
thank you Frédéric. Paul |
comment:10
Some of these examples yield slightly different results on 32-bit systems:
|
comment:11
On Linux ia64:
|
comment:12
the new attachment should fix the numerical noise issues on both 32-bit systems and ia64. Paul |
comment:13
Replying to @zimmermann6:
The |
comment:14
Hello, I am not happy with the indentation: it should be 4 spaces instead of 2 ! Otherwise, this should be good to go. |
comment:15
Attachment: trac_14318_doctests_from_french_book.patch.gz
fixed in the new attachment. Paul |
comment:16
There still remains
|
Commit: |
Branch: u/chapoton/14318 |
comment:18
Paul or Laurent, if you agree with my changes, you can set this to positive review. |
comment:20
Yes, the first commit is just the patch trac_14318_doctests_from_french_book.patch turned into a git commit |
Changed author from Laurent Fousse, Paul Zimmermann to Laurent Fousse, Paul Zimmermann, Frédéric Chapoton |
Changed reviewer from Frédéric Chapoton to Frédéric Chapoton, Paul Zimmermann |
comment:21
thanks, then I give a positive review. Paul |
The attached file provides doctests for examples in the chapter on integration and differential equations. Tests pass with 5.11.
Apply only attachment: trac_14318_doctests_from_french_book.patch
Component: doctest coverage
Author: Laurent Fousse, Paul Zimmermann, Frédéric Chapoton
Branch/Commit: u/chapoton/14318 @
91f4bcb
Reviewer: Frédéric Chapoton, Paul Zimmermann
Issue created by migration from https://trac.sagemath.org/ticket/14318
The text was updated successfully, but these errors were encountered: