Skip to content
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

Implicit broadcasting is still in the docs #28569

Closed
mlhetland opened this issue Aug 10, 2018 · 2 comments
Closed

Implicit broadcasting is still in the docs #28569

mlhetland opened this issue Aug 10, 2018 · 2 comments
Labels
docs This change adds or pertains to documentation good first issue Indicates a good issue for first-time contributors to Julia help wanted Indicates that a maintainer wants help on an issue or pull request

Comments

@mlhetland
Copy link
Contributor

From the docs

Most of the binary arithmetic operators listed above also operate elementwise when one argument is scalar: -, +, and * when either argument is scalar, and / and \ when the denominator is scalar. For example, [1, 2] + 3 == [4, 5] and [6, 4] / 2 == [3, 2].

This is no longer true in 1.0. I didn't immediately have any obvious replacement – hence the issue and not a PR. I guess it ought to be replaced with a discussion of dotted operators, but the dot syntax is discussed in the following paragraphs, as an additional concept, so I guess a slight rewrite might be called for?

@JeffBezanson JeffBezanson added the docs This change adds or pertains to documentation label Aug 10, 2018
@StefanKarpinski StefanKarpinski added help wanted Indicates that a maintainer wants help on an issue or pull request good first issue Indicates a good issue for first-time contributors to Julia labels Aug 14, 2018
@timziebart
Copy link
Contributor

timziebart commented Aug 29, 2018

Actually, I just read through that section and I would suggest to simply remove that paragraph. Then the whole section makes sense and is consistent. Shall I do that and submit a PR?

Edit: The "Additionally" in the following paragraph needs to be removed, too.

@timziebart
Copy link
Contributor

The corresponding PR is merged and this Issue can be closed. (Seemingly, I cannot do that)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation good first issue Indicates a good issue for first-time contributors to Julia help wanted Indicates that a maintainer wants help on an issue or pull request
Projects
None yet
Development

No branches or pull requests

4 participants