darktable-org / darktable-org/lua-scripts

Merge clear_GPS.lua and reset_GPS.lua into geoToolbox.lua

Aperta
#268 30 commenti 0 reazioni 1 assegnatario Vedi su GitHub

@wpferguson ci sta già lavorando.

Dal 2/8/2020.

dt5.4
Lingua principale
Lua
Stelle
219
Fork
142
Nessuna PR unita negli ultimi 30g

Descrizione

When I submitted a PR to add reset_GPS.lua @supertobi noted that it might be a good fit in geoToolbox.lua. I had just read through the geoToolbox.lua code and thought the same thing, so I closed the PR and decided to merge the code. clear_GPS.lua also manipulates image GPS information so that is also probably a good fit in geoToolbox.lua.

I opened this issue so that there would be a place to discuss the merge and how to arrive at a best fit.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.