On Monday 10 September 2007 18:21, Pablo De Napoli wrote:
> I could not read your modification to my patch since you've uploaded
> that as an hg bundle. I think that for trac the best would be to
> upload patches in plain text. That way, trac knows how to format it so
> that it can be easy read on the web
> (it would be nice to have something like trac but based on mercurial,
> rather than in subversion, but I doubt that such a program exists)

Yes, I did put it up as an hg bundle.  I agree these are annoying, but they do 
one thing nicely -- in that bundle are two patches which cleanly separates 
the patch you wrote and the modifications I made to it.  That separation is 
not such a big deal here, but in general I think it is nice which is why I 
did it this way.  Of course, you can apply this to a vanilla 2.8.4.1 with 'hg 
apply' (or whatever the hg_sage.<method> way is).  For your own plain text 
pleasure, my changes are attached.

I'd be interested to hear what other people have to say about proper trac 
etiquette with bundles and patches building on one another.

--
Joel

--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to sage-devel@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at http://groups.google.com/group/sage-devel
URLs: http://sage.scipy.org/sage/ and http://modular.math.washington.edu/sage/
-~----------~----~----~----~------~----~------~--~---

# HG changeset patch
# User Joel B. Mohler <[EMAIL PROTECTED]>
# Date 1189455866 25200
# Node ID 6baf1b85853190a600968642018e79e560148ccf
# Parent  8def8d03e4a2edc4544cad18b931a706c2f8e0fe
Minor documentation changes for binomial global function on top of Pablo De Napoli's patch.

diff -r 8def8d03e4a2 -r 6baf1b858531 sage/rings/arith.py
--- a/sage/rings/arith.py       Mon Sep 10 13:06:30 2007 -0700
+++ b/sage/rings/arith.py       Mon Sep 10 13:24:26 2007 -0700
@@ -1858,21 +1858,19 @@ def binomial(x,m):
     $$
        x (x-1) \cdots (x-m+1) / m!
     $$
-    which is defined for $m \in \Z$ and any $x$.
-    If $m<0$ return $0$.
-
-    If x-m is an integer we define:
-
+    which is defined for $m \in \Z$ and any $x$.  We extend this definition to
+    include cases when $x-m$ is an integer but $m$ is not by
+
     binomial(x,m)= binomial(x,x-m)

-    (This rule makes sense if x is assumed to be an integer)
-
+    If $m<0$ return $0$.
+
     INPUT::
         x,m -- numbers or symbolic expressions
-        Either x or x-m must be an integer.
+        Either m or x-m must be an integer.

     OUTPUT::
-        number
+        number or symbolic expression (if input is symbolic)

     EXAMPLES::
         sage: binomial(5,2)

Reply via email to