Opened 14 years ago
Closed 14 years ago
Last modified 14 years ago
#654 closed defect (fixed)
g.proj -c fails with ERROR: region for current mapset is not set run "g.region"
|Reported by:||hamish||Owned by:|
As reported on grass-user
g.proj -c georef=swilAlphaTIFF.tif location=tset
fails with recent builds. I can reproduce on linux.
I traced this back to r37726 where wind_format.c's format_double() uses G_projection() which wants to check the projection type (and that hasn't been created yet).
I think the solution is to create a new lib fn
char *G_format_number(double value, int dp);
so modules don't have to lie to G_format_northing() about their projection type in order to get numeric output from lat/lon locations. then we can get rid of G_projection() from format_double().
Either that or finally implement GRASS_DMS_STYLE variable and temporarily set that to numeric degree style.
Change History (8)
comment:1 by , 14 years ago
comment:2 by , 14 years ago
I think the solution is to create a new lib fn char *G_format_number(double value, int dp);
better, path of least change: allow G_format_northing() etc to accept "-1" for projection type which in this instance can mean use "%.15g" independent of the projection.
by , 14 years ago
patch to give G_format_*() full prec. fmt option
comment:3 by , 14 years ago
patch attached, but that's the easy part. It turns out a lot of modules are already using proj == -1 or similar to fool G_format_*() into outputting numerical output instead of DMS for certain tasks. i.e. this patch, or something like it, is long overdue.
need to audit all of the custom workarounds before applying..
comment:4 by , 14 years ago
comment:5 by , 14 years ago
|Status:||new → closed|
follow-up: 7 comment:6 by , 14 years ago
This seems to have fixed #648 also, at least for me applied to RC5 source on OSX.
Replying to hamish:
actually it has in a way,
the top two lines of PERMANENT/DEFAULT_WIND are written:
I'm not really sure if that helps though.