Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[3.12] gh-102837: more tests for the math module (GH-111930) #112029

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

Closed
wants to merge 1 commit into from

Conversation

skirpichev
Copy link
Contributor

@skirpichev skirpichev commented Nov 13, 2023

Add tests to improve coverage:

  • fsum: L1369, L1379, L1383, L1412
  • trunc: L2081
  • log: L2267
  • dist: L2577, L2579
  • hypot: L2632
  • sumprod: L2744, L2754, L2774, L2778, L2781, L2785, L2831, L2835, L2838
  • pow: L2982
  • prod: L3294, L3308, L3318-3330

// line numbers wrt to 9dc4fb8
(cherry picked from commit c61de45)

This include fsum tests from #102523

Add tests to improve coverage:

* fsum: L1369, L1379, L1383, L1412
* trunc: L2081
* log: L2267
* dist: L2577, L2579
* hypot: L2632
* sumprod: L2744, L2754, L2774, L2778, L2781, L2785, L2831, L2835, L2838
* pow: L2982
* prod: L3294, L3308, L3318-3330

// line numbers wrt to 9dc4fb8
(cherry picked from commit c61de45)

Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>
@encukou
Copy link
Member

encukou commented Nov 13, 2023

Oh, I just opened #112030 -- which backports BadDescr and some more tests.
Sorry for the duplicate work!

@skirpichev
Copy link
Contributor Author

@encukou, no - I'll close this. Your PR is in a better shape.

@skirpichev skirpichev closed this Nov 13, 2023
@skirpichev skirpichev deleted the backport-c61de45-3.12 branch November 13, 2023 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.