← 返回首页
Use normalized rng data for Large* tests of pinverse by 9prady9 · Pull Request #3152 · arrayfire/arrayfire · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

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

Provide feedback

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

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
Resetting focus

Use normalized rng data for Large* tests of pinverse#3152

Merged
9prady9 merged 1 commit into
arrayfire:masterfrom
9prady9:pinverse_test_fix
Jun 24, 2021
Merged

Use normalized rng data for Large* tests of pinverse#3152
9prady9 merged 1 commit into
arrayfire:masterfrom
9prady9:pinverse_test_fix

Conversation

Copy link
Copy Markdown
Member

9prady9 commented Jun 22, 2021

Description

Float type has accuracy issues with large input values which is the case with current test data for Large & LargeTall test inputs.

Changes to Users

None

Checklist

  • Rebased on latest master
  • Code compiles
  • Tests pass
  • [ ] Functions added to unified API
  • [ ] Functions documented

9prady9 added this to the 3.8.1 milestone Jun 22, 2021
9prady9 requested a review from umar456 June 22, 2021 11:23
Float type has accuracy issues with large input values for pinverse computations. This change updates the data sets for Large & LargeTall tests that has this accuracy issue.
9prady9 force-pushed the pinverse_test_fix branch from 737568d to 01f9e4b Compare June 23, 2021 10:01
syurkevi self-requested a review June 24, 2021 03:06
9prady9 merged commit 2a2b677 into arrayfire:master Jun 24, 2021
9prady9 deleted the pinverse_test_fix branch June 24, 2021 03:09
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Footer

© 2026 GitHub, Inc.