Opened 8 years ago

Last modified 5 years ago

#3150 new enhancement

Simple r.out.png region enhancement

Reported by: huhabla Owned by: grass-dev@…
Priority: trivial Milestone: 7.6.2
Component: Raster Version: svn-trunk
Keywords: r.out.png region Cc:
CPU: Unspecified Platform: Unspecified

Description

I would like to post a patch that enables direct region settings in r.out.png. This patch is only useful if the current region should not be changed or can not be changed due to access restrictions of the mapset.

Attachments (1)

r.out.png.patch (3.9 KB ) - added by huhabla 8 years ago.
r.out.png region patch

Download all attachments as: .zip

Change History (12)

by huhabla, 8 years ago

Attachment: r.out.png.patch added

r.out.png region patch

comment:1 by martinl, 8 years ago

Component: DefaultRaster
Milestone: 7.0.5

comment:2 by martinl, 8 years ago

Milestone: 7.0.57.2.0

Changing milestone at least to 7.2.0 (only bugfixes can go to relbr70).

comment:3 by neteler, 7 years ago

Milestone: 7.2.07.2.1

Ticket retargeted after milestone closed

comment:4 by martinl, 7 years ago

Milestone: 7.2.17.2.2

comment:5 by martinl, 7 years ago

Milestone: 7.2.27.4.0

All enhancement tickets should be assigned to 7.4 milestone.

comment:6 by neteler, 6 years ago

Milestone: 7.4.07.4.1

Ticket retargeted after milestone closed

comment:7 by neteler, 6 years ago

Milestone: 7.4.17.4.2

in reply to:  description comment:8 by wenzeslaus, 6 years ago

Keywords: region added

Replying to huhabla:

I would like to post a patch that enables direct region settings in r.out.png.

I agree that this would come handy time to time. For example, G7:r.mapcalc (>=7.4) has region option to customize the region based on the inputs. General modifications of region, such as the one you suggest, would apply to many modules.

I think this would be better addressed generally as we discussed in Adding an expert mode to the parser and Passing computational region to individual commands.

I suggest to close this as won't fix and open another ticket suggesting change on the library and parser level or on the level of grass.py covering the --exec case.

This patch is only useful if the current region should not be changed or can not be changed due to access restrictions of the mapset.

I was also in this situation and really for outputs and rendering, even many GUI operations, there is no need to have any write access or ownership (see also #2511). The current solution for this particular use case and r.out.png is the GRASS_REGION environmental variable.

comment:9 by martinl, 6 years ago

Milestone: 7.4.27.6.0

All enhancement tickets should be assigned to 7.6 milestone.

comment:10 by martinl, 5 years ago

Milestone: 7.6.07.6.1

Ticket retargeted after milestone closed

comment:11 by martinl, 5 years ago

Milestone: 7.6.17.6.2

Ticket retargeted after milestone closed

Note: See TracTickets for help on using tickets.