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.univar: fix MASK check with nprocs > 1 #4297

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

petrasovaa
Copy link
Contributor

With MASK openmp parallelization can crash. The check for MASK was not in the right place.

@petrasovaa petrasovaa added bug Something isn't working backport to 8.4 PR needs to be backported to release branch 8.4 labels Sep 9, 2024
@petrasovaa petrasovaa added this to the 8.4.1 milestone Sep 9, 2024
@petrasovaa petrasovaa self-assigned this Sep 9, 2024
@github-actions github-actions bot added raster Related to raster data processing C Related code is in C module labels Sep 9, 2024
@petrasovaa petrasovaa merged commit cc2797c into OSGeo:main Sep 9, 2024
26 of 27 checks passed
@petrasovaa petrasovaa deleted the fix-r.univar-mask-check branch September 9, 2024 18:38
@petrasovaa petrasovaa removed the backport to 8.4 PR needs to be backported to release branch 8.4 label Sep 9, 2024
@echoix echoix linked an issue Sep 10, 2024 that may be closed by this pull request
Mahesh1998 pushed a commit to Mahesh1998/grass that referenced this pull request Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working C Related code is in C module raster Related to raster data processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] r.univar in Grass84 (Windows) with qgis 3.38.2
2 participants