-
Notifications
You must be signed in to change notification settings - Fork 116
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
Random ranges #1038
Random ranges #1038
Conversation
eaddb70
to
6ff9aed
Compare
8d86d3e
to
0bd75fb
Compare
Unit tests are failing in Release mode only. Need to switch to Release configuration in Visual Studio when running the unit tests, in order to reproduce the issue. Looks like it is failing become ILGPU does not support |
Given the fact that this PR should not change the behavior of the ILGPU tests affected at the moment, this is particularly interesting. |
I think this is a change in the latest .NET runtimes - our nightly builds are failing too. |
This would actually make sense. If this problem is related to some .NET runtime updates, we should try to address this issue in v1.5.0. |
@MoFtZ as this blocks all CI jobs from completing at the moment, we have to fix it asap anyway. |
ef81e78
to
9240e5c
Compare
* Added new random ranges to create random numbers in certain intervals. * Added vector-specific random extensions.
* Added new random ranges to create random numbers in certain intervals. * Added vector-specific random extensions.
* Added new random ranges to create random numbers in certain intervals. * Added vector-specific random extensions.
This PR adds new
IRandomRange
andIRandomRangeProvider
interfaces to define common methods to combine random number generators with[min,.. max)
value ranges. It adds a text template to generateRandomRangeXYZ
types representing tuples ofmin
andmax
values with utility RNG functions. The template also generates provider implementations combining the tuples with actual RNG types likeRandomRangeXYZProvider
.This PR depends on PRs
#1033, #1037, and #1057.