Skip to content

Conversation

@nmaarnio
Copy link
Collaborator

No description provided.

Added pixel size and CRS as parameters. Added checks for inputs. Cancelling the saving no longer gives a warning.
@nmaarnio nmaarnio added the enhancement New feature or request label Apr 14, 2025
@nmaarnio nmaarnio linked an issue Apr 14, 2025 that may be closed by this pull request
@nmaarnio nmaarnio requested review from LKajan and Mtk112 April 22, 2025 12:05
@nmaarnio nmaarnio added the UI/UX label May 5, 2025
Copy link
Contributor

@LKajan LKajan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ryhti vaatii, että koordinaatisto on aina 3067, eli sitä käyttäjän ei tarvi ikinä valita itse.
Samoin en tiedä onko käyttäjän syytä valita edes pikselikokoa. Sen joka tapauksessa pitää ryhti-speksin mukaan olla jokin 2:n potenssi.
Kun käyttäjälle ei kauheasti jää valinnan varaa, niin onko lomakkeelle tarvetta?

Lisäksi mieluusti pitäisin niin paljon sovelluslogiikasta gui-tiedostojen ulkopuolella kun mahdollista. Ton geotiffin luonnin voisi oikeastaan siirtää qgisin background taskiksi (jos vaan mahdollista), niin siinäkin auttaisi, että se on toteutettu omana moduulinaan.

@nmaarnio
Copy link
Collaborator Author

Okei, sitten poistetaan ainakin koordinaatiston valinta. Itse ajattelin, että tuotettavan kuvan resoluution valinta olisi tärkeä olla mukana, mutta voidaan sekin ottaa pois.

Ehkä oma lomake ei ole välttämätön, mutta mielestäni tämä toiminto ei ainakaan tarvitse omaa nappia toolbarissa. Vie paljon tilaa, ja toolbarissa on tyypillisemmin maptoolien, lomakkeiden ja dockwidgettien nappeja, eikä suoraan prosesseja.

Kaavan tallentaminen JSONina ja GeoTIFFinä voisivat ainakin elää yhdessä, joko yhteisessä lomakkeessa tai valikkona toolbarin napin alla. Mietin myös, pitäisikö kaavan vieminenkin laittaa samaan yhteyteen.

@nmaarnio nmaarnio marked this pull request as draft May 15, 2025 11:21
@LKajan
Copy link
Contributor

LKajan commented May 16, 2025

Totta, että resoluutiota täytyy jollain tapaa säätää. Pienellä asemakaavalla on varmasti eri resoluutio kuin maakuntakaavalla. Mutta en tiedä tuleeko tää antaa käyttäjän säädettäväksi (Ainakin voisi jotenkin pakottaa käyttämään noita 2:n potensseja).

Tuo voisi olla ihan hyvä idea, että on yksi "Export"-nappi ja sitten voi valita kaava-json, kaava-asia-json, geotiff, tuloste (geopdf). Näin esim pgModeler toimii.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Luo ikkuna Tallenna kaavakartta -toiminnolle

3 participants