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.import: use temp region for resolution=value #197

Merged
merged 1 commit into from
Nov 19, 2019

Conversation

wenzeslaus
Copy link
Member

Temp region was not used when g.region was called with resolution and -a.
Use same temp region settings as for extent=input.
Alignment (g.region res= -a) added in ff0cece.
This surfaces, e.g., when running r.import in parallel.

Temp region was not used when g.region was called with resolution and -a.
Use same temp region settings as for extent=input.
@wenzeslaus wenzeslaus self-assigned this Nov 15, 2019
@landam
Copy link
Member

landam commented Nov 15, 2019

Candidate for backport?

@wenzeslaus wenzeslaus merged commit fceba69 into OSGeo:master Nov 19, 2019
@wenzeslaus wenzeslaus deleted the fix-r-import-no-temp-region branch November 19, 2019 19:01
petrasovaa pushed a commit to petrasovaa/grass that referenced this pull request Dec 3, 2019
Temp region was not used when g.region was called with resolution and -a.
Use same temp region settings as for extent=input.
neteler pushed a commit that referenced this pull request Dec 23, 2019
Temp region was not used when g.region was called with resolution and -a.
Use same temp region settings as for extent=input.
@neteler
Copy link
Member

neteler commented Dec 23, 2019

Backported to relbranch78 in 889bf2b, hoping to address #250

landam pushed a commit to landam/grass that referenced this pull request Jan 28, 2020
Temp region was not used when g.region was called with resolution and -a.
Use same temp region settings as for extent=input.
@neteler neteler added this to the 7.8.2 milestone Dec 9, 2021
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.

None yet

4 participants