Skip to content
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

r.circle: Add tests for raster circle tool #4598

Merged
merged 20 commits into from
Nov 8, 2024

Conversation

Shreshth-Malik
Copy link
Contributor

@Shreshth-Malik Shreshth-Malik commented Oct 26, 2024

This PR adds unit tests for the r.circle module. The tests include:

Binary Circle Test: Verifies the creation of an output and a binary raster map using the -b flag.
Distance Circle Test: Checks the output of an output and a distance-based circle raster without the -b flag.
Multiplier flag test: Checks the output of an output after giving a multiplier as input.

@github-actions github-actions bot added raster Related to raster data processing Python Related code is in Python module tests Related to Test Suite labels Oct 26, 2024
@Shreshth-Malik Shreshth-Malik changed the title R.circle test r.circle test Oct 26, 2024
@Shreshth-Malik Shreshth-Malik changed the title r.circle test r.circle: Add tests for raster circle tool Oct 26, 2024
@petrasovaa petrasovaa added this to the 8.5.0 milestone Nov 1, 2024
Shreshth-Malik and others added 2 commits November 2, 2024 11:52
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

@petrasovaa petrasovaa left a comment

Choose a reason for hiding this comment

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

Please also test the multiplier option.

@petrasovaa petrasovaa merged commit b18e390 into OSGeo:main Nov 8, 2024
26 checks passed
@Shreshth-Malik Shreshth-Malik deleted the r.circle-test branch November 8, 2024 19:56
a0x8o pushed a commit to a0x8o/grass that referenced this pull request Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module Python Related code is in Python raster Related to raster data processing tests Related to Test Suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants