-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Fixed parenthesis error #15003
Fixed parenthesis error #15003
Conversation
Since both of these evaluate to the same (correct) value, it's unclear to me that this was an error: julia> -(3/4*im)
-0.0 - 0.75im
julia> -(3/4)*im
-0.0 - 0.75im |
Isn't there some extra/missing minus signs going on here? |
NVM, it is correct. |
I didn't notice the precence of the *-operator, because the text is about the literal coefficient and the division. |
Leave the *-operator out would make the example more clear to me: |
I updated the patch. 3/4im == 3/(4im) == -(3/4)im I had read the previous equation (in a mathematical sense) as 3/4im == -(3/4im) which seemed to be an error to me. |
At this point this PR is just a cosmetic change which I don't think conveys significant value. However, some additional wording about paying attention to the parentheses in the expression would make a useful contribution. |
It seems a bit clearer, let's just merge it. |
Fixed parenthesis error
Note that this was against the release branch. It's generally preferable to open PR's against master first. |
No description provided.