Skip to content

Refine CFL computation for Q/P polynomials#26

Merged
haraldkl merged 16 commits intomainfrom
cfl
Nov 13, 2025
Merged

Refine CFL computation for Q/P polynomials#26
haraldkl merged 16 commits intomainfrom
cfl

Conversation

@haraldkl
Copy link
Copy Markdown
Member

@haraldkl haraldkl commented Nov 7, 2025

@haraldkl haraldkl self-assigned this Nov 7, 2025
@haraldkl haraldkl requested a review from skn96 November 7, 2025 15:55
@haraldkl
Copy link
Copy Markdown
Member Author

haraldkl commented Nov 7, 2025

@skn96 I'll look into this next week, it looks like we may have to pass some additional argument, or compute the nPoly differently in the caller.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 10, 2025

Waf Unit Tests

  2 files    2 suites   0s ⏱️
102 tests 102 ✅ 0 💤 0 ❌
204 runs  204 ✅ 0 💤 0 ❌

Results for commit 06af97e.

♻️ This comment has been updated with latest results.

@haraldkl haraldkl marked this pull request as ready for review November 10, 2025 14:09
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 11, 2025

System Tests

58 tests   58 ✅  25m 20s ⏱️
58 suites   0 💤
58 files     0 ❌

Results for commit 06af97e.

♻️ This comment has been updated with latest results.

@haraldkl
Copy link
Copy Markdown
Member Author

@skn96 this is now ready to be merged from my point of view if you don't have any objections.

Copy link
Copy Markdown
Contributor

@skn96 skn96 left a comment

Choose a reason for hiding this comment

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

Hi Harald, I also have no problems with merging the changes!

@haraldkl haraldkl merged commit b933b1f into main Nov 13, 2025
3 checks passed
@haraldkl haraldkl deleted the cfl branch November 13, 2025 11:44
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