Skip to content

Use TestSuite for projections tests - #125

Merged
kshyatt merged 3 commits into
mainfrom
testsuite-projections
Dec 23, 2025
Merged

Use TestSuite for projections tests#125
kshyatt merged 3 commits into
mainfrom
testsuite-projections

Conversation

@kshyatt

@kshyatt kshyatt commented Dec 22, 2025

Copy link
Copy Markdown
Member

What it says on the tin. Use TestSuite for the projections tests and additionally test them with some new element types!

@kshyatt
kshyatt requested review from Jutho and lkdvos December 22, 2025 18:23
@kshyatt
kshyatt force-pushed the testsuite-projections branch from 4b17bbc to 33b61d4 Compare December 22, 2025 20:13
Comment thread test/testsuite/projections.jl Outdated
@kshyatt
kshyatt force-pushed the testsuite-projections branch from 33b61d4 to e5b3073 Compare December 23, 2025 06:21
@kshyatt
kshyatt enabled auto-merge (squash) December 23, 2025 06:55
@codecov

codecov Bot commented Dec 23, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kshyatt
kshyatt merged commit 4212b74 into main Dec 23, 2025
10 checks passed
@kshyatt
kshyatt deleted the testsuite-projections branch December 23, 2025 07:49
# test approximate error calculation
A = normalize!(A)
normA = norm(A)
A ./= normA

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is normalize! another GPU problem?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes 😭

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll try to add support for it to GPUArrays.jl

@lkdvos lkdvos mentioned this pull request Dec 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants