Check results are now reported in the status checks at the bottom of this page.

Codecov Report

Merging #8108 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #8108      +/-   ##
==========================================
+ Coverage   86.92%   86.92%   +<.01%     
==========================================
  Files         383      383              
  Lines       57821    57822       +1     
  Branches     1056     1056              
==========================================
+ Hits        50262    50263       +1     
  Misses       6945     6945              
  Partials      614      614

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c29ecd...83412a2. Read the comment docs.

This will definitely need tests, or fixes to the current tests to make them fail on the previous version.

Thanks @ycopin ! I will be able to look at this in a couple of days.

@bsipocz I think this is failing because there's no section for 3.1.1 in CHANGES. How should this be handled? Should I make one?

I think the tests for derivatives did not fail because of the specific values used in the tests - gamma=1, x_0=0, y_0=0. While those tests are OK, we should add a more general one.

@nden - don't worry about the changelog, I'll sort it out once the release is really out. Since this is a bugfix, I suppose it can land in RC2 once you merge it.

I guess the bot is then confused because there is a changelog entry and it has the correct PR number.

The PR number in the changelog is incorrect (it has to be the PR's number not the issue's it fixes), but I can fix it while doing the release, feel free to merge whenever this is ready.

@nden thanks for the review.

To test the derivatives, 0s and 1s are always susceptible to be special values... Just systematically take 0.5 or 2 or whatever else!