Skip to content

Add new kernels - #3486

Merged
pomadchin merged 3 commits into
locationtech:masterfrom
yang162132:add_new_kernels
Apr 19, 2025
Merged

pomadchin merged 3 commits into
locationtech:masterfrom
yang162132:add_new_kernels

Conversation

@yang162132

@yang162132 yang162132 commented Oct 28, 2022 •

Copy link
Copy Markdown
Contributor

Overview

Add some new kernel for focal calculate.

Checklist

  • ./CHANGELOG.md updated, if necessary. Link to the issue if closed, otherwise the PR.
  • Unit tests added for bug-fix or new feature

@pomadchin pomadchin left a comment

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.

Hello! Thanks for a fantastic contribution, and sorry for a very dealyed review.

Could I ask you to adjust tests? So tests actually compare the computed result with some expected values.

@pomadchin pomadchin mentioned this pull request Apr 19, 2025

@pomadchin pomadchin left a comment

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.

Thanks for your contribution; sorry it took a bit of time to sort it out!

@pomadchin
pomadchin merged commit 922cfda into locationtech:master Apr 19, 2025
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