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

Improve docs for abs, abs2 #45141

Merged
merged 6 commits into from Aug 7, 2022
Merged

Improve docs for abs, abs2 #45141

merged 6 commits into from Aug 7, 2022

Conversation

mcabbott
Copy link
Contributor

@mcabbott mcabbott commented May 1, 2022

This adds some examples for abs, abs2, since someone in this thread started at the beginning of the alphabet...

base/number.jl Outdated Show resolved Hide resolved
base/int.jl Outdated Show resolved Hide resolved
base/number.jl Outdated Show resolved Hide resolved
base/number.jl Outdated Show resolved Hide resolved
@dkarrasch dkarrasch added the doc This change adds or pertains to documentation label May 2, 2022
mcabbott and others added 2 commits May 2, 2022 08:57
Co-authored-by: Steven G. Johnson <stevenj@mit.edu>
@ViralBShah
Copy link
Member

ViralBShah commented Jun 19, 2022

@mcabbott Is this ready from your perspective? I feel the comments are adequately addressed and I would be comfortable to merge.

@ViralBShah
Copy link
Member

@KristofferC You had some concerns above - I feel this is good to merge and overall makes the documentation more accessible.

@mcabbott
Copy link
Contributor Author

Sorry I missed your earlier comment. From my side this seems OK; also fine to delete the maximum(abs, [1, -2, 3, -4]) example if this seems too far afield.

ViralBShah and others added 2 commits July 16, 2022 13:55
Co-authored-by: Michael Abbott <32575566+mcabbott@users.noreply.github.com>
@ViralBShah
Copy link
Member

CI is on a hiatus at the moment, but if we get a pass on doctest, let's merge.

base/number.jl Outdated Show resolved Hide resolved
@ViralBShah ViralBShah merged commit 649de67 into JuliaLang:master Aug 7, 2022
@mcabbott mcabbott deleted the doc5 branch August 8, 2022 04:33
ffucci pushed a commit to ffucci/julia that referenced this pull request Aug 11, 2022
Co-authored-by: Steven G. Johnson <stevenj@mit.edu>
Co-authored-by: Viral B. Shah <ViralBShah@users.noreply.github.com>
pcjentsch pushed a commit to pcjentsch/julia that referenced this pull request Aug 18, 2022
Co-authored-by: Steven G. Johnson <stevenj@mit.edu>
Co-authored-by: Viral B. Shah <ViralBShah@users.noreply.github.com>
LilithHafner added a commit that referenced this pull request Feb 12, 2024
This extends the docs for `Int`, `Float64` etc, to
* Note that Int is the default, mention that it can overflow
* Explain what `1f0` means, and that Float64 is the default

Similarly extends docs for `^`, `+`, `*` aiming to
* Point out that `1.2 * 10^3` is a bad habit, warn about overflow
* Give an example for what `literal_pow` is doing since the explanation
is quite technical
* Also point out overflow in `+`, as [suggested
here](#45141 (comment))
* While there, mention that you can add vectors, and that vararg
`+(1,2,3,4)` has a default binary behaviour
* Similarly mention that vararg `*(1,2,3,4)` has a default order, from
the left
* While there, mention `1/2pi` and `v'v` as examples of `*`.

---------

Co-authored-by: Lilith Orion Hafner <lilithhafner@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants