-
Notifications
You must be signed in to change notification settings - Fork 2
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
Generating integers #35
Labels
Comments
Changed the field elements generator in 967a8b0 to Field -> frequency
[ (2, fromIntegral <$> arbitrarySizedBoundedIntegral @Int)
, (1, arbitrary)
] Now we produce integer-valued field elements pretty often. Not sure why we didn't do this earlier. |
Made it Field -> frequency
[ (1, fromIntegral <$> arbitrary @Int)
, (1, fromIntegral <$> arbitrarySizedBoundedIntegral @Int)
, (1, arbitrary)
] in b02791e so that fairly small integers also get generated. |
Whenever we generate |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Previously we only needed to generate integers in comparisons, but now we have plenty of places where it would be useful to generate integers. I'm now getting test failures only with the
F17
field, because theArbitrary
instance ofRational
is just extremely unlikely to produce an integer at all:This begs to be resolved.
The text was updated successfully, but these errors were encountered: