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

Error (on Windows) when moving rasters in certain projections #7

Open
gioman opened this issue Dec 18, 2014 · 10 comments
Open

Error (on Windows) when moving rasters in certain projections #7

gioman opened this issue Dec 18, 2014 · 10 comments

Comments

@gioman
Copy link

gioman commented Dec 18, 2014

Con questa modifca

https://github.com/geodrinx/rasmover/pull/5/files#diff-7e06dfda38ebac72bfc7a550603b78a7R228

(nella PR che ho chiuso) avevo "risolto" il problema del file "moved.vrt" che non viene correttamente creato sotto Linux. In un secondo momento poi mi sono accorto, sotto Windows, che uno stesso raster in un certo sistema di coordinate viene spostato correttamente, mentre in un altro avviene un errore python. La cosa strana é che sotto Linux, con lo stesso raster non ci sono problemi in nessun sistema di coordinate. L'errore su Windows é del tipo

Traceback (most recent call last):
File "C:/Users/giovanni/.qgis2/python/plugins\rasmover\rasmover.py", line 218, in canvasReleaseEvent
Y = float( coordY ) + dy
ValueError: invalid literal for float(): 0.0000000000000000e+000-2.0347502734837728e+000

Ho quindi capito da cosa é dovuto.

Quando il raster ha certi sistemi di coordinate, il primo valore nel tag

< GeoTransform >

puó essere negativo, per esempio:

< GeoTransform > -9.0608624274958577e+05...

il problema é che sotto Windows il comando gdalbuildvrt non mette lo spazio prima del "-", e il tag diventa

< GeoTransform >-9.0608624274958577e+05...

e questo fa in modo che poi

words = stringa.split();

non funzioni correttamente, perché per fare lo split di default é usato lo spazio.

Questo problema di mancanza di spazio avviene (su Windows) anche tra il penultimo e ultimo valore del tag < GeoTransform >, esempio

Linux:
... 0.0000000000000000e+00, -2.0347502734837728e+00</ GeoTransform>

Windows:
...0.0000000000000000e+00,-2.0347502734837728e+00</ GeoTransform>

e questo fa in modo (di nuovo) che lo split della stringa non funzioni allo stesso modo nei due sistemi operativi.

Questo secondo problema di mancanza di spazio (che crea il problema della creazione di moved.vrt non corretto su Linux) si puó risolvere aggiungendo

              if _platform != "win32":
                 stringazza = ("%s %s ") %( stringazza, words[6] )

mas ovviamente risolvendo alla radice il problema dello split della stringa risolverebbe tutto in un colpo solo.

@geodrinx
Copy link
Owner

OK Giovanni :)
Quindi, se ho capito bene, avresti risolto il problema. Giusto ?
Poi, come avevo intuito, si trattava anche di un problema di coordinate ? Vero ? Grande !
Ma quindi, in Portogallo le coordinate sono anche negative ? E già: state a ovest di Greenwich ...
No ?

Comunque, tu puoi inserire la correzione le ramo principale della repository github ?
Devo abilitarti ? Non sono un grande esperto di github. Cosa devo fare ? Ovviamente, inseriamo i nostri nomi nella testata del file rasmover.py ...

Ciao e grazie
Roberto

@gioman
Copy link
Author

gioman commented Dec 18, 2014

Con questa pull request/patch

#8

ho risolto i due problemi: 1) installazione su Linux 2) problema di creazione di moved.vrt/parsing della stringa con il datasource in Linux/Windows.

Per provare usa rasmover come é in questo momento con questo raster

https://www.dropbox.com/s/es5spl8py4k9d2b/googlegeo.tif?dl=0

dopo sostituisci rasmover.py con questa copia (con le modifiche)

https://github.com/gioman/rasmover/blob/master/rasmover.py

@gioman
Copy link
Author

gioman commented Dec 18, 2014

riesci a rilascare una nuova versione del plugin con le correzzioni?

@geodrinx
Copy link
Owner

Giovanni,

riesci a rilascare una nuova versione del plugin con le correzzioni?

:) a parte la parola "correzzioni" con qualche zeta di troppo
(scherzzzzo ovviamente :)

sto provando a caricare il plugin (con il tuo nome, anche :) nella
repository di QGIS, ma mi ritorna un

"This field is required."

senza specificare quale campo ... :(

Non capisco se è stato inserito o no. Tu riesci a controllare ?

Fammi sapere

Ciao

Roberto


Reply to this email directly or view it on GitHub
#7 (comment).

@gioman
Copy link
Author

gioman commented Dec 19, 2014

Parlo e scrivo 3 lingue diverse tutti i giorni, e da emigrato da 12 anni credo che qualche errore me lo posso permettere ;)

Nella mia PR aggiornata

https://github.com/geodrinx/rasmover/pull/8/files

ho corretto il file metadata.txt e aggiunto le cartelle mancanti (in particolare "temp").

Quando qualcuno fa delle correzzioni usando una Pull Request, e queste sono a posto, ti consiglio di fare il merge accettando la stessa PR. Credo che le mie prime correzioni tu le abbia aggiunte manualmente, e ora che ne ho fatte altre la PR non si puó mergiare automaticamente e ti toccherá aggiungerle di nuovo manualmente. Comunque il nuovo metadata.txt credo sia a posto e credo che ti permetterá l'upload della nuova versione.

@gioman
Copy link
Author

gioman commented Dec 19, 2014

Il file metadata.txt lo puoi scaricare da qua

https://github.com/gioman/rasmover/blob/master/metadata.txt

@geodrinx
Copy link
Owner

Nella mia PR aggiornata

https://github.com/geodrinx/rasmover/pull/8/files

ho corretto il file metadata.txt e aggiunto le cartelle mancanti (in
particolare "temp").

Quando qualcuno fa delle correzzioni usando una Pull Request, e queste
sono a posto, ti consiglio di fare il merge accettando la stessa PR. Credo
che le mie prime correzioni tu le abbia aggiunte manualmente, e ora che ne
ho fatte altre la PR non si puó mergiare automaticamente e ti toccherá
aggiungerle di nuovo manualmente. Comunque il nuovo metadata.txt credo sia
a posto e credo che ti permetterá l'upload della nuova versione.

Hai sicuramente ragione, ma io non riesco a vedere la tua modifica. C'è
da dire che io non sono affatto esperto di github (ma sono in buona
compagnia, perchè molti colleghi lo odiano :)

Domanda: perchè la versione precedente veniva caricata ? Posso sfogarmi
con uno strillo ? Tappati gli occhi (per non dire gli orecchi) :

SI PUO' SAPERE PER QUALE CAVOLO DI MOTIVO SI CAMBIANO INUTILMENTE LE COSE ?

grazie, ora mi sento meglio. ;) Tu non c'entri assolutamente nulla, ma
mi ci voleva.
Mi sfogherò presto anche meglio sul mio blog :)

Comunque, dove posso prendere lo ZIP che va caricato correttamente ?

Ciao e grazie

Roberto

@gioman
Copy link
Author

gioman commented Dec 19, 2014

Ciao Roberto

le modifiche sono qua:

https://github.com/geodrinx/rasmover/pull/8/files

  1. aggiunto un "README" vuoto dentro i18n, perché cartelle vuote con git non si possono committare
  2. aggiunto un ".gitignore" dentro "temp", per la stessa ragione di 1)
  3. modificato un commento in "rasmover.py"
  4. modificato metadata.txt
  5. rimossi informazioni non pertinenti dal README

Git é complicato (ma necessario), ma non é impossibile e soprattutto le cosa da sapere/imparare per gestire i propri plugin sono alla fine 3/4.

"Domanda: perchè la versione precedente veniva caricata ?" -> non ho capito a cosa ti riferisci

"SI PUO' SAPERE PER QUALE CAVOLO DI MOTIVO SI CAMBIANO INUTILMENTE LE COSE ?" -> ti riferisci a qgis?

@geodrinx
Copy link
Owner

Il giorno 19 dicembre 2014 11:34, Giovanni Manghi [email protected]
ha scritto:

Ciao Roberto

le modifiche sono qua:

https://github.com/geodrinx/rasmover/pull/8/files

  1. aggiunto un "README" vuoto dentro i18n, perché cartelle vuote con git
    non si possono committare
  2. aggiunto un ".gitignore" dentro "temp", per la stessa ragione di 1)
  3. modificato un commento in "rasmover.py"
  4. modificato metadata.txt
  5. rimossi informazioni non pertinenti dal README

Git é complicato (ma necessario), ma non é impossibile e soprattutto le
cosa da sapere/imparare per gestire i propri plugin sono alla fine 3/4.

Sono d'accordo. Diciamo che github non è proprio intuitivo, ma poi
diventa facile. Solo, potevano fare dei bottoni gerarchici
sull'interfaccia del client e non ... dei bottoni posizionati in maniera
qualunque di grandezze qualunque che non si capisce quando devono essere
premuti... :)

"Domanda: perchè la versione precedente veniva caricata ?" -> non ho
capito a cosa ti riferisci

"SI PUO' SAPERE PER QUALE CAVOLO DI MOTIVO SI CAMBIANO INUTILMENTE LE COSE
?" -> ti riferisci a qgis?

Lascia perdere lo sfogo di chi ha dovuto (penso come te) modificare 4 volte
il codice, su altri plugin, per andare appresso a ( per me inutili )
modifiche nelle api di pyQgis. Mi chiedo: perchè modificare e non
aggiungere funzioni ? Chi paga lo sviluppo (e la modifica) di plugin
funzionanti per "allinearsi" a ... non si sa cosa ? :) Ma vabbè.

A presto

Roberto


Reply to this email directly or view it on GitHub
#7 (comment).

@gioman
Copy link
Author

gioman commented Dec 19, 2014

capisco il problema con client github per windows/mac, a me pare molto piú semplice dare quei 2/3 comandi dalla linea di comandi (su linux o quant'altro), sono veramente semplici. Sui cambiamenti di qgis la vedo cosí: qgis ha (aveva) bisogno di "recuperare" tanto terreno sugli "altri" e per questo le cose cambiano (sono cambiate) rapidamente, ora si é arrivati ad un punto in cui la 2.8 sará finalmente sufficientemente matura e stabile al punto da poterne fare una LTS, ossia una versione che rimarrá tale a lungo. Riesci a pubblicare la nuova versione del plugin? alcune persone ne avrebbero bisogno (e senza le modifche gli da errore su Windows per via delle coordinate)? grazie e buon natale!

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

No branches or pull requests

2 participants