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

Heatmap style #76

Closed ma15569 closed 1 year ago

ma15569 commented 1 year ago
PLUS-install-raster_tools wget https://sourceforge.net/projects/opensit/files/Openjump/PlugIn/Raster%20Tools/RasterTools-2.0.3-28112022.jar/download 17443c1aa8e0b7e67d50f3c8da49c9a0c70b77f6 false ${plus.folder}
edeso commented 1 year ago

yeah. no. that looks wrong. you seem to have based your changes on some outdated 'heatmap' branch.

better start with branch main of a current clone/fork. read here e.g. https://www.freecodecamp.org/news/how-to-make-your-first-pull-request-on-github-3/ they do it step by step .

ma15569 commented 1 year ago

I Ede, the info you sent me seems available only for Linux ( I actually use windows). Anyhow I don't understand if the pull request I did some hours ago was Ok or not:

[image: immagine.png]

Il giorno mer 30 nov 2022 alle ore 14:20 edeso @.***> ha scritto:

Closed #76 https://github.com/openjump-gis/openjump/pull/76.

— Reply to this email directly, view it on GitHub https://github.com/openjump-gis/openjump/pull/76#event-7922019159, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQSYPM46Y6PRETDZI2SFNQTWK5IBNANCNFSM6AAAAAASOWVT3E . You are receiving this because you authored the thread.Message ID: @.***>

edeso commented 1 year ago

@ma15569 well the steps are generally the same, on any platform. it was just the first example i could find. fell free to search for yourself ;)

wrt. to your other PR. i added comments in there and renamed to a speaking title. please read and adapt there.

ma15569 commented 1 year ago

Ok. I will try with GitHub Desktop, tomorrow Peppe

Il giorno mer 30 nov 2022 alle ore 17:22 edeso @.***> ha scritto:

@ma15569 https://github.com/ma15569 well the steps are generally the same, on any platform. it was just the first example i could find. fell free to search for yourself ;)

wrt. to your other PR https://github.com/openjump-gis/openjump/pull/77. i added comments in there and renamed to a speaking title. please read and adapt there.

— Reply to this email directly, view it on GitHub https://github.com/openjump-gis/openjump/pull/76#issuecomment-1332421200, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQSYPM36VPYYT3G3NS7HVQDWK55NJANCNFSM6AAAAAASOWVT3E . You are receiving this because you were mentioned.Message ID: @.***>

edeso commented 1 year ago

well,

normally you would use probably Eclipse and egit (Eclipse git integration), but as the changes are fairly simple you can do them online as well. just go to your branch that is connected to the Pull Request https://github.com/ma15569/openjump/tree/patch-10

click the file you want to edit, then on the right top there is a 'Pencil icon' (edit mode) edit commit et'voila ;)

good luck! ..ede

On 30.11.2022 17:25, ma15569 wrote:

Ok. I will try with GitHub Desktop, tomorrow Peppe

Il giorno mer 30 nov 2022 alle ore 17:22 edeso @.***> ha scritto:

@ma15569 https://github.com/ma15569 well the steps are generally the same, on any platform. it was just the first example i could find. fell free to search for yourself ;)

wrt. to your other PR https://github.com/openjump-gis/openjump/pull/77. i added comments in there and renamed to a speaking title. please read and adapt there.

— Reply to this email directly, view it on GitHub https://github.com/openjump-gis/openjump/pull/76#issuecomment-1332421200, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQSYPM36VPYYT3G3NS7HVQDWK55NJANCNFSM6AAAAAASOWVT3E . You are receiving this because you were mentioned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/openjump-gis/openjump/pull/76#issuecomment-1332425351, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACMDSQL6X67S5AGYRASY2SDWK55ZJANCNFSM6AAAAAASOWVT3E. You are receiving this because you modified the open/close state.Message ID: @.***>