[GRASS-dev] grass.jupyter Mini Project: Week 3 Recap

2022-02-01 Thread Caitlin Haedrich
Hello grass-dev and grass-users,

Here's a quick update on my Mini Project for improving grass.jupyter. Per
usual, you can find more details about the project and a copy of this
report at [1].

*1) What did I accomplish this week?*

   - Switched from a Dropdown menu to a SelectionSlider for the time_slider
   visualization
  - I found a bug in ipywidgets [2] which slowed me down a bit this
  week. I ended up opening an issue on GitHub and it was quickly addressed
  (go ipywidgets!).
   - Wrote TimeSeries.animate() function for animating space time datasets

You can test it out on Binder at [3] and view the PR at [4].


*2) What do I plan to do next week?*

   - Cleanup and write nice docstrings for TimeSeries visualizations
   - The current visualizations are not great for multitemporal datasets
   (i.e. datasets with varying time steps), since the steps in the animation
   and time slider are all regularly spaced. Next week, I'll work on a way to
   address this.
   - Maybe start working on GRASS-folium integration for interactive mapping


*3) Am I blocked on anything?*

   - No, I'm not currently blocked on anything.

Feedback, thoughts and comments welcome!

Have a great week,
Caitlin

[1]
https://trac.osgeo.org/grass/wiki/GSoC/2021/JupyterAndGRASS/MiniGrant2022
[2] https://github.com/jupyter-widgets/ipywidgets/issues/3363
[2]
https://mybinder.org/v2/gh/chaedri/grass/Jupyter-Timeseries?urlpath=lab%2Ftree%2Fdoc%2Fnotebooks%2FTemporal.ipynb
[3] https://github.com/OSGeo/grass/pull/2010
___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] new addon

2022-02-01 Thread Francesco Geri

Thank you very much.

Cloning my fork everything gone ok.

I created a new branch called "r.forcircular", add the new folder and 
then commit/push. Then I did the pull request at the website.


I hope I did everything right this time.

Thanks again.


Il 01/02/22 12:41, Ondřej Pešek ha scritto:
út 1. 2. 2022 v 12:40 odesílatel Francesco Geri > napsal:


/git clone g...@github.com:fra-lab/grass.git
/

///Clone in 'grass' in corso...//
//The authenticity of host 'github.com 
(140.82.121.3)' can't be established.//
//ECDSA key fingerprint is
SHA256:p2QAMXNIC1TJYWeIOttrVc98/R1BUFWu3/LiyKgUfQM.//
//Are you sure you want to continue connecting
(yes/no/[fingerprint])? yes//
//Warning: Permanently added 'github.com
,140.82.121.3' (ECDSA) to the list of known
hosts.//
//g...@github.com : Permission denied
(publickey).//
//fatal: Impossibile leggere dal repository remoto.//
//Assicurati di disporre dei privilegi d'accesso corretti//
//e che il repository esista./

In english:

/fatal: Unable to read from remote repository.
Make sure you have the correct access privileges
and that the repository exists.
/


Did you fork the repository before trying to clone it? I cannot see 
your fork in the list of forks and also not on your profile when 
stalking it.


You can do it interactively at the website by going to [1] or [2] 
(depending on whether you want to fork GRASS itself, or the addons 
repository) and clicking on "Fork" in the upper right corner. If you 
didn't do that, then the repository fra-lab/grass does not exist and 
therefore couldn't be cloned.


[1] https://github.com/OSGeo/grass 
[2] https://github.com/OSGeo/grass-addons 

___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] new addon

2022-02-01 Thread Stefan Blumentrath
Hi Francesco,

And welcome!
Just a little heads up: The right place to add a new module is the grass-addons 
repository (you seem to try to add it to the GRASS GIS core):
grass-addons/CONTRIBUTING.md at grass8 * OSGeo/grass-addons 
(github.com)

Please note that there are also some formal, "legal boxes" to tick:
"
The submission must be compliant with the GRASS submission rules as found in 
the GRASS source code and RFC2 (Legal aspects of submission):
https://trac.osgeo.org/grass/wiki/RFC
"
If I remember correctly, you have to confirm towards the Project Steering 
Committee (PSC) that RFC 2 is read and understood and that your contributions 
adhere to the requirements outlined there...

Others, please correct me if that only applies to write access to the repo...

Looking forward to seeing your addon in the repository!

Cheers,
Stefan


From: grass-dev  On Behalf Of Ondrej Pešek
Sent: tirsdag 1. februar 2022 12:41
To: Francesco Geri 
Cc: GRASS developers list 
Subject: Re: [GRASS-dev] new addon

út 1. 2. 2022 v 12:40 odesílatel Francesco Geri 
mailto:francescog...@tim.it>> napsal:
git clone 
g...@github.com:fra-lab/grass.git

Clone in 'grass' in corso...
The authenticity of host 
'github.com
 (140.82.121.3)' can't be established.
ECDSA key fingerprint is SHA256:p2QAMXNIC1TJYWeIOttrVc98/R1BUFWu3/LiyKgUfQM.
Are you sure you want to continue connecting (yes/no/[fingerprint])? yes
Warning: Permanently added 
'github.com,140.82.121.3'
 (ECDSA) to the list of known hosts.
g...@github.com: Permission denied (publickey).
fatal: Impossibile leggere dal repository remoto.
Assicurati di disporre dei privilegi d'accesso corretti
e che il repository esista.

In english:

fatal: Unable to read from remote repository.
Make sure you have the correct access privileges
and that the repository exists.

Did you fork the repository before trying to clone it? I cannot see your fork 
in the list of forks and also not on your profile when stalking it.

You can do it interactively at the website by going to [1] or [2] (depending on 
whether you want to fork GRASS itself, or the addons repository) and clicking 
on "Fork" in the upper right corner. If you didn't do that, then the repository 
fra-lab/grass does not exist and therefore couldn't be cloned.

[1] 
https://github.com/OSGeo/grass
[2] 
https://github.com/OSGeo/grass-addons
___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] new addon

2022-02-01 Thread Ondřej Pešek
út 1. 2. 2022 v 12:40 odesílatel Francesco Geri 
napsal:

> *git clone g...@github.com:fra-lab/grass.git
> *
>
> *Clone in 'grass' in corso...*
> *The authenticity of host 'github.com  (140.82.121.3)'
> can't be established.*
> *ECDSA key fingerprint is
> SHA256:p2QAMXNIC1TJYWeIOttrVc98/R1BUFWu3/LiyKgUfQM.*
> *Are you sure you want to continue connecting (yes/no/[fingerprint])? yes*
> *Warning: Permanently added 'github.com ,140.82.121.3'
> (ECDSA) to the list of known hosts.*
> *g...@github.com : Permission denied (publickey).*
> *fatal: Impossibile leggere dal repository remoto.*
> *Assicurati di disporre dei privilegi d'accesso corretti*
> *e che il repository esista.*
>
> In english:
>
>
>
>
> *fatal: Unable to read from remote repository. Make sure you have the
> correct access privileges and that the repository exists.*
>

Did you fork the repository before trying to clone it? I cannot see your
fork in the list of forks and also not on your profile when stalking it.

You can do it interactively at the website by going to [1] or [2]
(depending on whether you want to fork GRASS itself, or the addons
repository) and clicking on "Fork" in the upper right corner. If you didn't
do that, then the repository fra-lab/grass does not exist and therefore
couldn't be cloned.

[1] https://github.com/OSGeo/grass
[2] https://github.com/OSGeo/grass-addons
___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] new addon

2022-02-01 Thread Francesco Geri

Thank you very much Markus

If the CONTRIBUTING.md file isn't clear in this regard, please let us
know which line(s) should be improved. A great occasion to make it
better!


in fact, I have difficulty in the very initial stages of set-up. When I 
try to follow the line to fork the repository:


/git clone g...@github.com:your_GH_account/grass.gi/t

and so

/git clone g...@github.com:fra-lab/grass.git/

I obtain this error message:

/git clone g...@github.com:fra-lab/grass.git//
//Clone in 'grass' in corso...//
//The authenticity of host 'github.com (140.82.121.3)' can't be 
established.//
//ECDSA key fingerprint is 
SHA256:p2QAMXNIC1TJYWeIOttrVc98/R1BUFWu3/LiyKgUfQM.//

//Are you sure you want to continue connecting (yes/no/[fingerprint])? yes//
//Warning: Permanently added 'github.com,140.82.121.3' (ECDSA) to the 
list of known hosts.//

//g...@github.com: Permission denied (publickey).//
//fatal: Impossibile leggere dal repository remoto.//
//Assicurati di disporre dei privilegi d'accesso corretti//
//e che il repository esista./

In english:

/fatal: Unable to read from remote repository.
Make sure you have the correct access privileges
and that the repository exists.
/



___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev


Re: [GRASS-dev] new addon

2022-02-01 Thread Markus Neteler
Hi Francesco,

On Tue, Feb 1, 2022 at 9:53 AM Francesco Geri  wrote:
>
> Hello everybody.
> I am writing to the grass-dev list because I am having difficulty to
> upload a new add-on called r.forcircuar to the Grass addon repository.
> The add-on is called r.forcircular, it was developed by University of
> Florence inside a research project aiming to the circular bioeconomy and
> sustainability of the forest supply chain.

Great!

> I followed the procedure shown at this link
> https://github.com/OSGeo/grass/blob/main/CONTRIBUTING.md , but every
> time I try to push I get an error message related to incorrect access
> privileges. My github account is fra-lab.

The idea is that you open a pull request so that it can be easily
reviewed. Upon approval it will then be merged.

If the CONTRIBUTING.md file isn't clear in this regard, please let us
know which line(s) should be improved. A great occasion to make it
better!

> Thank you very much for helping.
> Francesco

Best,
Markus
___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev


[GRASS-dev] new addon

2022-02-01 Thread Francesco Geri

Hello everybody.
I am writing to the grass-dev list because I am having difficulty to 
upload a new add-on called r.forcircuar to the Grass addon repository. 
The add-on is called r.forcircular, it was developed by University of 
Florence inside a research project aiming to the circular bioeconomy and 
sustainability of the forest supply chain. I followed the procedure 
shown at this link 
https://github.com/OSGeo/grass/blob/main/CONTRIBUTING.md , but every 
time I try to push I get an error message related to incorrect access 
privileges. My github account is fra-lab.


Thank you very much for helping.
Francesco


___
grass-dev mailing list
grass-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/grass-dev