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

t.rast.algebra: do not print when debug is off #2926

Merged
merged 1 commit into from
Apr 22, 2023

Conversation

petrasovaa
Copy link
Contributor

Also fixes some related minor issues.

@petrasovaa petrasovaa added the bug Something isn't working label Apr 19, 2023
@petrasovaa petrasovaa added this to the 8.3.0 milestone Apr 19, 2023
@veroandreo
Copy link
Contributor

veroandreo commented Apr 21, 2023

I tested with the examples in the studio at NCSU. The assessment of conditions per map are no longer printed (Yay!), but now I get a bunch of these: WARNING:root:Raster range file is empty that didn't appear before.

These are the commands:

t.rast.extract input=lst_daily where="start_time <= '2014-06-30'" output=test basename=test suffix=gran
t.rast.algebra expression="algebra_test = if(test >= 20.0 && test <= 30.0, 1, null())" basename=algebra_test suffix=gran nproc=7 -n

IIUC, the warning refers to the null maps that I keep on purpose using the n flag. If that's the case, why the warning? Perhaps it's unrelated, but it has only appeared now. If I do not use the n flag, then I get both the warnings per map and a message (also per map) informing that it is removed. The warning is even ignoring the superquiet flag --qq

@veroandreo
Copy link
Contributor

I tested again in freshly compiled 8.3 without this PR, and the warning is also printed then, so it's unrelated to this PR (though kinda annoying imho). I tested also in 8.2.0 and the warning is not printed there, so something changed in between that produces the warning.

+1 for merging this one then, thanks Anna!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants