-
Notifications
You must be signed in to change notification settings - Fork 364
Release 410k-2 #7179
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
Merged
Release 410k-2 #7179
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
+pow performance degrades rapidly as the exponent grows. Here we replace the +sqt implementation that calls +pow with one that uses the Newton-Raphson method. This gives way better performance, and more accurate results to boot. Before: ```hoon > ~> %bout (sqt:rd:lm .~1e-2) .~0.10000000022017597 :: took ms/181.095 > ~> %bout (sqt:rd:lm .~1e-3) .~0.03162277732556293 :: took s/10.576.252 > ~> %bout (sqt:rd:lm .~1e-4) .~0.010000002308497298 :: took s/743.265.918 > ~> %bout (sqt:rd:lm .~1e2) .~9.999999977324386 :: took ms/181.242 > ~> %bout (sqt:rd:lm .~1e3) .~31.622775866034136 :: took s/10.573.055 ``` After: ```hoon > ~> %bout (sqt:rd:lm .~1e-2) .~0.09999999999999999 :: took µs/80 > ~> %bout (sqt:rd:lm .~1e-3) .~0.03162277660168379 :: took µs/78 > ~> %bout (sqt:rd:lm .~1e-4) .~0.01 :: took µs/105 > ~> %bout (sqt:rd:lm .~1e2) .~10 :: took µs/114 > ~> %bout (sqt:rd:lm .~1e3) .~31.622776601683793 :: took µs/73 ```
Implements +asin, +acos and +atan. The former two are implemented in terms of the latter, which is implemented as a Taylor expansion. We also include the two-argument variant of +atan, +atan2, for completeness.
To keep the numericists at ease.
Merge master back to develop
ames: add %tune to $sign to not break old queued-events
The runtime makes the http requests for double boot protection using the standard C `%d` format but the endpoint in eyre expects the bone to be `@ud` formatted. This causes a mismatch if the ping-bone is larger than 1000, causing double boot protection to become disabled. We address the mismatch in arvo instead of vere because this is easier to release.
math: better +sqt with newton's method
math: inverse trigonometric functions
eyre: fix double boot protection bone format
#7126 but for develop
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.