On Sun, 11 Aug 2024 07:30:12 GMT, Chris Hennick wrote:
>> This provides a slightly more accurate bounding limit for
>> `computeNextExponentialSoftCapped`. Currently, if the maximum is specified
>> as 12.0, it won't actually return a value larger th
https://github.com/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with three additional
commits since the last revision:
- Lower limit to fix timeout
- Fix?
- Revert "Fix?"
https://github.com/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Fix?
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17703/files
-
by
> https://github.com/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with five additional
commits since the last revision:
- Merge remote-tracking branch 'origin/master' into patch-2
- Merge pu
by
> https://github.com/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Fix?
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17703/files
-
On Sat, 10 Aug 2024 03:00:10 GMT, Chris Hennick wrote:
>> This provides a slightly more accurate bounding limit for
>> `computeNextExponentialSoftCapped`. Currently, if the maximum is specified
>> as 12.0, it won't actually return a value larger th
om/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Fix: must add 1 *after* dividing
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17703/files
- new: https://git.openjdk.
om/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Fix a syntax error
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17703/files
- new: https://git.openjdk.org/jdk/pu
om/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Fix a test bug
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17703/files
- new: https://git.openjdk.org/jdk/pull/17703/
om/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Test with larger powers of 2
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17703/files
- new: https://git.openjdk.
On Wed, 12 Jun 2024 09:35:12 GMT, Jaikiran Pai wrote:
>> Chris Hennick has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Bug fix: add-exports was for wrong package
>
> test/jdk/jdk/internal/util/random/
On Wed, 12 Jun 2024 08:16:25 GMT, Jaikiran Pai wrote:
>> Chris Hennick has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Bug fix: add-exports was for wrong package
>
> src/java.base/share/cl
om/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Address review comments
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17703/files
- new: https://git.openjdk.org/jdk/pu
On Wed, 12 Jun 2024 09:23:23 GMT, Jaikiran Pai wrote:
>> Chris Hennick has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Bug fix: add-exports was for wrong package
>
> test/jdk/jdk/internal/util/random
om/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Update RandomSupportTest.java
-
Changes:
- all: https://git.openjdk.org/jdk/pull/17703/files
- new: https://git.openjdk.
On Wed, 12 Jun 2024 10:08:02 GMT, Jaikiran Pai wrote:
>>> Update: confirmed that the new test fails without the fix.
>>
>> Thanks for verifying the test checks the fix; I'll let others who have
>> worked more directly on the random code review the actual fix.
>
>> @jddarcy Added a regression te
om/openjdk/jdk/pull/17703/commits/b8be051cbf40a6a05fafc6a2c76942e9e0b11fdf.
Chris Hennick has updated the pull request with a new target base due to a
merge or a rebase. The pull request now contains five commits:
- Merge branch 'openjdk:master' into patch-2
- Bug fix: add-exports was for wrong package
- Tweak bounda
On Wed, 21 Feb 2024 02:18:08 GMT, Chris Hennick wrote:
>> This provides a slightly more accurate bounding limit for
>> `computeNextExponentialSoftCapped` when calling it from
>> `computeNextGaussian`. This could cause the `while
>> (computeNextExponentialSoftCapped(rng
On Wed, 21 Feb 2024 02:18:08 GMT, Chris Hennick wrote:
>> This provides a slightly more accurate bounding limit for
>> `computeNextExponentialSoftCapped` when calling it from
>> `computeNextGaussian`. This could cause the `while
>> (computeNextExponentialSoftCapped(rng
On Wed, 21 Feb 2024 02:18:08 GMT, Chris Hennick wrote:
>> This provides a slightly more accurate bounding limit for
>> `computeNextExponentialSoftCapped` when calling it from
>> `computeNextGaussian`. This could cause the `while
>> (computeNextExponentialSoftCapped(rng
On Wed, 21 Feb 2024 02:18:08 GMT, Chris Hennick wrote:
>> This provides a slightly more accurate bounding limit for
>> `computeNextExponentialSoftCapped` when the computed bound is greater than
>> `(1.0p53 - 1.0) * DoubleZigguratTables.exponentialX0`. This could ca
On Wed, 21 Feb 2024 02:18:08 GMT, Chris Hennick wrote:
>> This provides a slightly more accurate bounding limit for
>> `computeNextExponentialSoftCapped` when the computed bound is greater than
>> `(1.0p53 - 1.0) * DoubleZigguratTables.exponentialX0`. This could ca
On Tue, 20 Feb 2024 20:46:47 GMT, Joe Darcy wrote:
>> This provides a slightly more accurate bounding limit for
>> `computeNextExponentialSoftCapped` when the computed bound is greater than
>> `(1.0p53 - 1.0) * DoubleZigguratTables.exponentialX0`. This could cause the
>> `while (computeNextExp
imit)` check in
> `computeNextGaussian` on line 1402 to always be true, making the
> `nextGaussian` runtime unbounded in the worst case.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Bug fix: add-exports was for wrong package
-
imit)` check in
> `computeNextGaussian` on line 1402 to always be true, making the
> `nextGaussian` runtime unbounded in the worst case.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Tweak boundary cases and add unit test
-
On Mon, 5 Feb 2024 04:25:16 GMT, Chris Hennick wrote:
> This provides a slightly more accurate bounding limit for
> `computeNextExponentialSoftCapped` when the computed bound is greater than
> `(1.0p53 - 1.0) * DoubleZigguratTables.exponentialX0`. This could cause the
This provides a slightly more accurate bounding limit for
`computeNextExponentialSoftCapped` when the computed bound is greater than
`(1.0p53 - 1.0) * DoubleZigguratTables.exponentialX0`. This could cause the
`while (computeNextExponentialSoftCapped(rng, limit) < limit)` check in
`computeNextGa
On Wed, 6 Apr 2022 17:47:53 GMT, Chris Hennick wrote:
> This PR improves both the worst-case performance of `nextExponential` and
> `nextGaussian` and the distribution of output at the tails. It fixes the
> following imperfections:
>
> * Repeatedly adding DoubleZigguratTables.e
m going into the `nextExponential` tail twice.
Chris Hennick has updated the pull request with a new target base due to a
merge or a rebase. The pull request now contains 21 commits:
- Merge branch 'openjdk:master' into patch-1
- Merge remote-tracking branch 'upstream/master' i
On Fri, 14 Jul 2023 20:42:03 GMT, Joe Darcy wrote:
>> @JimLaskey @jddarcy @turbanoff @rgiulietti Could this please be merged?
>
> @Pr0methean , please update your branch with the current contents of the
> master before issuing the integrate command. Assuming that succeeds as
> expected, myself
On Mon, 15 May 2023 15:51:02 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
On Mon, 15 May 2023 15:51:02 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
On Mon, 15 May 2023 15:51:02 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
m going into the `nextExponential` tail twice.
Chris Hennick has updated the pull request with a new target base due to a
merge or a rebase. The pull request now contains 20 commits:
- Merge remote-tracking branch 'upstream/master' into patch-1
- Merge remote-tracking branch 'or
On Thu, 6 Apr 2023 18:07:29 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
m going into the `nextExponential` tail twice.
Chris Hennick has updated the pull request incrementally with two additional
commits since the last revision:
- Merge remote-tracking branch 'origin/patch-1' into patch-1
- Optimize: move some code out of the fast path
-
Change
On Thu, 6 Apr 2023 18:07:29 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
m going into the `nextExponential` tail twice.
Chris Hennick has updated the pull request with a new target base due to a
merge or a rebase. The pull request now contains 17 commits:
- Merge branch 'master' of https://git.openjdk.org/jdk into patch-1
- Merge branch 'master' of
m going into the `nextExponential` tail twice.
Chris Hennick has updated the pull request with a new target base due to a
merge or a rebase. The pull request now contains 16 commits:
- Merge branch 'master' of https://git.openjdk.org/jdk into patch-1
- Merge branch 'master' i
m going into the `nextExponential` tail twice.
Chris Hennick has updated the pull request with a new target base due to a
merge or a rebase. The incremental webrev excludes the unrelated changes
brought in by the merge/rebase. The pull request contains 15 additional commits
since the last rev
On Wed, 22 Mar 2023 19:23:37 GMT, Eirik Bjorsnos wrote:
>> Chris Hennick has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Add parameter to enable/disable fixed PRNG seed
>
> Chris, copyrights in your new (20
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
On Wed, 7 Dec 2022 20:25:53 GMT, Xin Liu wrote:
>> It's probably just a statistical fluke involving a very large return value,
>> but to know for sure I'd need a way to gather statistics about the return
>> values and how they correlate with running-time outliers.
>
>> It's probably just a stat
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the worst-case performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
>> * Repeatedly adding DoubleZiggura
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
>> * Repeatedly adding DoubleZiggura
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
>> * Repeatedly adding DoubleZiggura
On Tue, 4 Oct 2022 17:36:56 GMT, Chris Hennick wrote:
>> This PR improves both the performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
>> * Repeatedly adding DoubleZiggura
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Add parameter to enable/disable fixed PRNG seed
-
Changes:
- all: https://git.openjdk.org/jdk/pull/8131
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request with a new target base due to a
merge or a rebase. The incremental webrev excludes the unrelated changes
brought in by the merge/rebase. The pull request contains ten additional
comm
On Sun, 19 Jun 2022 23:38:36 GMT, Chris Hennick wrote:
>> This PR improves both the performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
>> * Repeatedly adding DoubleZ
On Sun, 19 Jun 2022 23:38:36 GMT, Chris Hennick wrote:
>> This PR improves both the performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
>> * Repeatedly adding DoubleZ
On Sun, 19 Jun 2022 23:38:36 GMT, Chris Hennick wrote:
>> This PR improves both the performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
>> * Repeatedly adding DoubleZ
On Sun, 19 Jun 2022 23:38:36 GMT, Chris Hennick wrote:
>> This PR improves both the performance of `nextExponential` and
>> `nextGaussian` and the distribution of output at the tails. It fixes the
>> following imperfections:
>>
>> * Repeatedly adding DoubleZ
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Rewrite Javadoc
-
Changes:
- all: https://git.openjdk.org/jdk/pull/8131/files
- new: https://git.op
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Simplify Javadoc description
-
Changes:
- all: https://git.openjdk.org/jdk/pull/8131/files
- new: http
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Weaken contract of computeWinsorizedNextExponential to save max/min calls
-
Changes:
- all: https://gi
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Revert "Bugfix: need a thread-local instance for throughput test"
This reverts commit d903efe57225e916
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has refreshed the contents of this pull request, and previous
commits have been removed. The incremental views will show differences compared
to the previous content of the PR. The pull request contains one new c
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
More sensible benchmark-mode settings
-
Changes:
- all: https://git.openjdk.org/jdk/pull/8131/files
-
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Bugfix: need a thread-local instance for throughput test
-
Changes:
- all: https://git.openjdk.org/jdk/
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has refreshed the contents of this pull request, and previous
commits have been removed. The incremental views will show differences compared
to the previous content of the PR. The pull request contains one new co
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Split out PRNG benchmarks into org.openjdk.bench.java.util.random
-
Changes:
- all: https://gi
Gaussian` suffers from *two* tail
> cases of `nextExponential`.
Chris Hennick has updated the pull request incrementally with one additional
commit since the last revision:
Add JMH
-
Changes:
- all: https://git.openjdk.org/jdk/pull/8131/files
- new: https://git.openjdk.org
67 matches
Mail list logo