#15346: Symbolic sums should evaluate
-------------------------+-------------------------------------------------
Reporter: | Owner:
kcrisman | Status: needs_review
Type: | Milestone: sage-6.4
defect | Resolution:
Priority: major | Merged in:
Component: | Reviewers: Karl-Dieter Crisman
symbolics | Work issues:
Keywords: | Commit:
Authors: Ralf | cb78603878bc1a7443e982824e13c81e22e9edda
Stephan | Stopgaps:
Report Upstream: N/A |
Branch: |
u/rws/15346 |
Dependencies: |
#17731 |
-------------------------+-------------------------------------------------
Comment (by mjo):
Ok, I think I see what happened. False alarm. Looking at Ralf's original
commit in comment 6, he added `simplify_sum` to `simplify_full`, but that
was before I removed `simplify_log` and the redundant `simplify_rational`.
In comment 10, the git bot merges with develop, pulling in the patch that
removed `simplify_log` and the redundant `simplify_rational`. But then in
the commit from comment 16, the `simplify_log` and `simplify_rational` are
back and git thinks they're being re-added.
I'm not sure exactly where things went awry. At this point I usually throw
my hands up, `git reset --hard`, and then copy/paste paste the code back
where it's supposed to go. You can force-push the branch and check the
link above to make sure it's not re-adding `simplify_log`.
--
Ticket URL: <http://trac.sagemath.org/ticket/15346#comment:25>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
