-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
unXFAIL two tests in test_wester #9242
Conversation
def test_V12(): | ||
r1 = integrate(1/(5 + 3*cos(x) + 4*sin(x)), x) | ||
# Correct result in python2.7.4 wrong result in python3.3.1 | ||
# https://github.com/sympy/sympy/issues/7157 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about leaving in a reference to #7157 so if this regresses, people know where to look for more information?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
@toolforger Are you +1 now? |
I'd have left in the reference to the issue report. |
@skirpichev Was fad74a8 reverted? |
The tests are run with random seeds so if it is still an issue it should show up. |
It passed once. Should I restart it? |
I don't see this commit in the history. |
Me too but then why it has |
No idea. Perhaps, this is commit in my fork (now deleted). |
From the old messages, it seemed to me that the randomness has just selects one or the other of two equally valid results. Since we haven't seen wrong results, we probably can still just run the test as it is and revisit the issue if we ever see a problem. A few more test runs (locally, I guess) should give you enough confidence that it won't happen too soon on Travis, or for anybody who runs the tests locally. |
Looks as arbitrary numbers for me... (BTW, how many words and how little sense.) @debugger22, try something like this (works fine for me):
Maybe some stuff in heurisch still depends on dict/set's sorting. Maybe not, i.e. after 2e06e1a (I have failure for above command before this commit.). I bet it was fixed by @smichr ;) |
It fails.
As @asmeurer already pointed out, it's independent of Python version too. Perhaps it can be closed. |
Oh, it seems I have found this "source of randomness". |
Am 12.04.2015 um 01:06 schrieb Sergey B Kirpichev:
Yeah. Arbitrary numbers are often a stand-in for free variables. So:
Trolling taken to private mail. |
def test_V12(): | ||
r1 = integrate(1/(5 + 3*cos(x) + 4*sin(x)), x) | ||
# Correct result in python2.7.4 wrong result in python3.3.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW, there is no "wrong result". I get tan(x/2)/(2*tan(x/2) + 4)
, that's same antiderivative as expr below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. The comment is misleading. It was already pointed out on the issue.
Closes #7157