openjump-gis / openjump

OpenJUMP, the Open Source GIS with more than one trick in its kangaroo pocket, takes the leap from svn to git. join the effort!
http://openjump.org
GNU General Public License v2.0
28 stars 14 forks source link

Added capability to RasterImage layer to read geogeaphic info from .aux.xml file #97

Closed ma15569 closed 1 month ago

ma15569 commented 1 year ago

See mail https://sourceforge.net/p/jump-pilot/mailman/message/37833644/ for more details

jratike80 commented 1 year ago

I believe (not sure) that aux.xml files may exist even without georeferencing info. Does the code skip aux.xml in such case and search for world files etc.?

edeso commented 1 year ago

hey Jukka,

it's pretty convoluted by now. but yeah, in theory it should try all methods until one succeeds. should be this part. try geotiff or if not avail, try sidecar file.

https://github.com/openjump-gis/openjump/blob/main/src/org/openjump/core/ccordsys/utils/ProjUtils.java#L468-L499

ma15569 commented 1 year ago

Hi Ede, this request seems still in progress.

edeso commented 1 year ago

@ma15569 just busy myself. give me some more time to review your changes please. :)

ma15569 commented 1 year ago

😕

edeso commented 1 year ago

thx for the reminder. just travelling.

will try to fit in a review while riding the train Mo or Tue :)

ma15569 commented 1 year ago

Hi Ede,

a) I removed the code org.openjump.core.ccordsys.AuxFile-srs=SRS se incluye en un archivo auxiliar as from the Spanis lang file as it is not used anymore Meanwhile I added new translations to that file

b) AdddRasterIageLayerWizard. I correct the typo from "slimed" to "optimized".

Not sure if the corrections are visible on main brench

Peppe

ma15569 commented 1 year ago

??

edeso commented 1 year ago

this line in ProjUtils

private static final String AUXFILE_SRS = I18N.getInstance().get("org.openjump.core.ccordsys.AuxFile-srs");

is obsolete now, is it not?

edeso commented 1 month ago

is this still ongoing or was it resolved inbetween?

ma15569 commented 1 month ago

Hi Ede, I prefer to abandon this request and focus on developing my own external plugins, where I have more possibilities to manage the code. This option has however been added to my Raster Tools. which have a parallel system of importing raster data. If you don't mind, I will close the request.

edeso commented 1 month ago

If you don't mind, I will close the request.

sure. go ahead :)