Comment #28 on issue 1727 by ronan.l...@gmail.com: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
I think we should only worry about conflicts with builtins. However, re and
im could maybe be renamed to Re and Im for consistency with Abs, but, on
balance, I think
Updates:
Labels: -Priority-Critical -Milestone-Release0.7.0 -NeedsReview -asmeurer
Priority-Medium
Comment #27 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
The summation fix has been pushed in. Should this issue remain open for
the
Comment #26 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
The abs = Abs fix has been pushed in. Now I just need someone to review
the sum = summation fix (https://github.com/sympy/sympy/pull/78).
--
You received this message because
Updates:
Labels: -Priority-High -Milestone-Release0.7.1 Priority-Critical
Milestone-Release0.7.0
Comment #25 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
(No comment was entered for this change.)
--
You received this message because
Comment #15 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
As mentioned on the mailing list, we also need to fix the builtin clash
with abs(). We should rename it to Abs(). I will create a patch for it
soon.
--
You received this
Comment #16 on issue 1727 by Vinzent.Steinberg: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
Abs() is inconsistent with any other sympy function. What about renaming
Sin(), Cos(), etc. too?
--
You received this message because you are subscribed to the Google
Comment #17 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
This was already suggested in issue 391. We can discuss if further, but
abs needs to be fixed before the release.
By the way, abs(expr) will still work because of __abs__.
--
Comment #18 on issue 1727 by Vinzent.Steinberg: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
Nice, I would do it this way for other functions too. So we could have
Sin() etc. for internal representation (as returned by srepr()) and the
user could still enter
Comment #19 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
I just noticed that we have min_() and max_(). But better would probably
be minimum() and maximum() or Min and Max(). I'm just noting it here; we
should worry about it after
Comment #20 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
See https://github.com/sympy/sympy/pull/79. Thanks to __abs__, abs() did
not have to be replaced with Abs in many cases. In fact, many files were
already using the built-in
Comment #21 on issue 1727 by smichr: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
eval_sum is not set up to handle indefinite sums as it assumes there are
always 3 items in the limits even though instantiatino allows 1, 2, or 3.
Other issues:
Sum doesn't
Comment #22 on issue 1727 by smichr: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
eval_sum is not set up to handle indefinite sums as it assumes there are
always 3 items in the limits even though instantiatino allows 1, 2, or 3.
Other issues:
Sum doesn't
Comment #23 on issue 1727 by smichr: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
I can confirm this:
set(['all', 'sum', '__doc__', 'abs', '__name__', '__package__', 'any'])
--
You received this message because you are subscribed to the Google Groups
Comment #24 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
Like I said, there are a ton of problems with Sum, which I conveniently
ignored for the moment :) We should create a new issue for them.
Is it foolish consistency then to see
Comment #10 on issue 1727 by Vinzent.Steinberg: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
I like summate(), which is a verb like integrate().
--
You received this message because you are subscribed to the Google Groups
sympy-issues group.
To post to this
Comment #11 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
summate() sounds awkward to me. ssum I do not like at all. I would rather
use a complete word when we can. summation() to me is even better than
sum() in some ways, because
Comment #12 on issue 1727 by Vinzent.Steinberg: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
summate() is unusual, but it makes the most sense to me (well, I'm not a
native speaker). But I'm also fine with summation(), which is indeed
consistent with product().
Comment #13 on issue 1727 by smichr: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
OK, I think the answer for this issue is already present in sympy: if you
can do the sum, do it, otherwise leave it as unevaluated and drop sum from
sympy. Here is the rationale:
Updates:
Labels: NeedsReview asmeurer
Comment #7 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
See https://github.com/sympy/sympy/pull/78.
--
You received this message because you are subscribed to the Google Groups
sympy-issues
Updates:
Cc: Vinzent.Steinberg Ronan.Lamy
Comment #8 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
Issue 1376 has been merged into this issue.
--
You received this message because you are subscribed to the Google Groups
Comment #9 on issue 1727 by fredrik.johansson: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
Maybe sums() or ssum()? It's short like sum_, without the ugly underscore,
and you can read it as symbolic sum / sum symbolically or SymPy sum /
sum with SymPy.
--
Updates:
Labels: -Priority-Medium -Milestone-Release0.7.0 Priority-High
Milestone-Release0.7.1
Comment #6 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
-1 for replacing sum() with Sum.doit() (because then it looks to the user
like
Comment #5 on issue 1727 by Vinzent.Steinberg: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
We could also remove sum() and replace it with Sum().doit() or
Sum().expand().
--
You received this message because you are listed in the owner
or CC fields of this issue,
Updates:
Blockedon: 1376
Comment #4 on issue 1727 by ronan.l...@gmail.com: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
FWIW, I don't really like 'summation', but it's the least bad of all the
names I
could think of.
--
You received this message because
Comment #2 on issue 1727 by Vinzent.Steinberg: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
Do wou think we could replace sum with Sum or Sum.doit?
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred
Comment #3 on issue 1727 by smichr: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
I thought I commented on this. FWIW, I like the summation suggestion of
Aaron's.
--
You received this message because you are listed in the owner
or CC fields of this issue, or
Status: Accepted
Owner: Vinzent.Steinberg
Labels: Type-Defect Priority-Medium Milestone-Release0.7.0
New issue 1727 by Vinzent.Steinberg: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
Sympy's sum for instance clashes with the built-in sum, maybe it should be
renamed
Updates:
Labels: EasyToFix
Comment #1 on issue 1727 by asmeurer: fix namespace clashes
http://code.google.com/p/sympy/issues/detail?id=1727
Or we could use Re and Im, since they are supposed to be inert classes when
they can't be evaluated anyway. I
strongly agree with this; these
28 matches
Mail list logo
