Skip to content

Conversation

@erdOne
Copy link
Contributor

@erdOne erdOne commented Dec 25, 2019

Brief description of what is fixed or changed

is_nthpow_residue(a, n, m) returns False when a % m == 0 and primitive_root(m) is not None but it should return True.

Release Notes

  • ntheory
    • Added check for zero in is_nthpow_residue.

is_nthpow_residue(a, n, m) returns False when a % m == 0 and primitive_root(m) is not None.
@sympy-bot
Copy link

sympy-bot commented Dec 25, 2019

Hi, I am the SymPy bot (v149). I'm here to help you write a release notes entry. Please read the guide on how to write release notes.

Your release notes are in good order.

Here is what the release notes will look like:

  • ntheory

This will be added to https://github.com/sympy/sympy/wiki/Release-Notes-for-1.6.

Note: This comment will be updated with the latest check if you edit the pull request. You need to reload the page to see it.

Click here to see the pull request description that was parsed.

<!-- Your title above should be a short description of what
was changed. Do not include the issue number in the title. -->

#### Brief description of what is fixed or changed
`is_nthpow_residue(a, n, m)` returns `False` when `a % m == 0 and primitive_root(m) is not None` but it should return `True`.

#### Release Notes

<!-- Write the release notes for this release below. See
https://github.com/sympy/sympy/wiki/Writing-Release-Notes for more information
on how to write release notes. The bot will check your release notes
automatically to see if they are formatted correctly. -->

<!-- BEGIN RELEASE NOTES -->
* ntheory
  * Added check for zero in is_nthpow_residue.
<!-- END RELEASE NOTES -->

Update

The release notes on the wiki have been updated.

@jksuom
Copy link
Member

jksuom commented Dec 25, 2019

There is (at least) one test that has to be fixed: assert is_nthpow_residue(2, 3, 2) is False intest_residue, otherwise this looks good.

@codecov
Copy link

codecov bot commented Dec 25, 2019

Codecov Report

Merging #18129 into master will increase coverage by <.001%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##            master    #18129       +/-   ##
=============================================
+ Coverage   74.933%   74.934%   +<.001%     
=============================================
  Files          642       642               
  Lines       166950    166970       +20     
  Branches     39275     39283        +8     
=============================================
+ Hits        125102    125118       +16     
+ Misses       36319     36313        -6     
- Partials      5529      5539       +10

@jksuom
Copy link
Member

jksuom commented Dec 26, 2019

Thank you for your contribution.

@jksuom jksuom merged commit cdf631e into sympy:master Dec 26, 2019
@erdOne erdOne deleted the patch-2 branch December 26, 2019 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants