Opened 4 years ago

Last modified 4 years ago

#3999 reopened enhancement

Mention projection of current location in error message

Reported by: jidanni Owned by: grass-dev@…
Priority: trivial Milestone: 7.8.3
Component: Projections/Datums Version: git-releasebranch78
Keywords: Cc:
CPU: Unspecified Platform: Unspecified

Description

$ grass --exec v.in.ogr -j input=e.geojson
WARNING: Projection of dataset does not appear to match current location.

         Dataset PROJ_INFO is:
         name: WGS 84
         datum: wgs84
         ellps: wgs84
         proj: ll
         no_defs: defined

         ERROR: proj

OK, it says

  • Dataset PROJ_INFO is:

so it also should say

  • Current location is:

even if the result is unknown (say so.) Yes even if the user forgot to run it inside grass.

Change History (6)

comment:1 by neteler, 4 years ago

Milestone: 7.8.3

comment:2 by neteler, 4 years ago

The projection non-/matching messages have been improved in https://github.com/OSGeo/grass/pull/317

Is it ok now?

comment:3 by jidanni, 4 years ago

Probably. I can't test for the next few months. So close this bug if it is OK now.

comment:4 by neteler, 4 years ago

Resolution: fixed
Status: newclosed

Ok, I'll close for now, feel free to reopen as needed with more tests.

in reply to:  4 comment:5 by mmetz, 4 years ago

Replying to neteler:

Ok, I'll close for now, feel free to reopen as needed with more tests.

It should be re-opened or referenced by a new issue on github because in this particular case, a XY location, nothing has changed in the messages, the location projection is not reported. Modules to be fixed are r.in.gdal, r.external, v.in.ogr, v.external.

comment:6 by neteler, 4 years ago

Resolution: fixed
Status: closedreopened
Note: See TracTickets for help on using tickets.