-
Notifications
You must be signed in to change notification settings - Fork 419
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
Rename BigInteger functions to camelCase and unstablize number theory functions #22788
Rename BigInteger functions to camelCase and unstablize number theory functions #22788
Conversation
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
459a75a
to
f49e2f2
Compare
Signed-off-by: Jade Abraham <[email protected]>
Signed-off-by: Jade Abraham <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one question about the unstable warning messages.
modules/standard/BigInteger.chpl
Outdated
:return: the Jacobi symbol | ||
:rtype: ``int`` | ||
*/ | ||
@unstable("jacobi is unstable and may move in the future") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is all that unstable means is that something may move in the future? I'm not sure we know for sure what might happen to any of these unstable methods, we just reserve the right to do whatever we want with them, so I think the wording "may move" sounds odd to me. Something like "may change" or "may be removed" or something makes more sense to me. What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree and I think I prefer "may change" since we reserve the right to do whatever we want with them
Signed-off-by: Jade Abraham <[email protected]>
Removes almost all of the remaining BigInteger deprecations This PR removes the deprecations from the following PRs I authored - #22775 - #22888/#22774 - #22818 - #22794 - #22788 - #22121 I also removed the remaining deprecations done by Yash Raj - #18855 - #18827 I moved one test out of the deprecated/BigInteger directory, `bigint_getlimbs.chpl`, since its not a deprecation test and actually checks functionality. Testing - [x] paratest without comm - [x] paratest with comm - [x] built docs and checked them [Reviewed by @ShreyasKhandekar]
Implements a number of renaming of functions in the BigInteger module to match the Chapel naming style. Also marks several functions as unstable that may move in the future.
Summary of changes
hamdist(a)
tohammingDistance(x)
, marked as unstablenextprime(result, a)
tonextPrime(result, x)
, marked as unstablelucnum
/lucnum2
tolucNum
/lucNum2
, marked as unstablejacobi
/legendre
/kronecker
as unstablegcd
/lcm
/fac
/bin
/fib
/fib2
/probablyPrime
as unstablegcdext
andprobab_prime_p
New tests
test/deprecated/BigInteger/deprecateHamDist.chpl
test/deprecated/BigInteger/deprecateNumberTheory.chpl
test/unstable/BigInteger/unstableHammingDistance.chpl
test/unstable/BigInteger/unstableNumberTheory.chpl
test/deprecated/BigInteger/deprecateGcdext.chpl
test/deprecated/BigInteger/deprecateProbabPrimeP.chpl
Testing
[Reviewed by @bmcdonald3]
closes #17733
partially implements decisions from #17724
marks functions in #17697 as unstable
closes cray/chapel-private#5098