Skip to content

Comments

i.evapo.time: Add example image to manual#3281

Merged
echoix merged 3 commits intomainfrom
YannChemin-patch-3
Dec 27, 2023
Merged

i.evapo.time: Add example image to manual#3281
echoix merged 3 commits intomainfrom
YannChemin-patch-3

Conversation

@YannChemin
Copy link
Contributor

Improved the explaination of the module with a figure and a reference

Improved the explaination of the module with a figure and a reference
@neteler
Copy link
Member

neteler commented Dec 2, 2023

@YannChemin: some notes:

  • the file i_evapo_time.png is missing (unfortunately the CI didn't catch that)
  • please (always) write PR titles to be ready for the automated ChangeLog creation using this style:
    module|topic: description
    Otherwise writing the release notes will be a pain. For PR title examples, see the entries e.g. here.
  • also set the milestone and relevant labels.

@echoix echoix changed the title Update i.evapo.time.html i.evapo.time: Add example image to manual Dec 23, 2023
@echoix echoix added manual Documentation related issues info needed Waiting on more info from the submitter labels Dec 23, 2023
@echoix
Copy link
Member

echoix commented Dec 23, 2023

If you need help, reach out to us!

@echoix echoix removed the info needed Waiting on more info from the submitter label Dec 27, 2023
Copy link
Member

@echoix echoix left a comment

Choose a reason for hiding this comment

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

I consider this a nice addition to improve the docs. If something is missing, it can be changed again later and doesn't regress the current state. I take the liberty to merge to have better docs available as soon as possible.

@echoix echoix merged commit 9b3979c into main Dec 27, 2023
@echoix echoix deleted the YannChemin-patch-3 branch December 27, 2023 15:18
neteler pushed a commit that referenced this pull request Dec 27, 2023
* Update i.evapo.time.html

Improved the explaination of the module with a figure and a reference

* Add example of temporal integration (#3280)

Ref: Chemin and Alexandridis 2004 https://ikee.lib.auth.gr/record/270217/files/Chemin-Alexandridis.pdf

---------

Co-authored-by: Edouard Choinière <[email protected]>
@neteler neteler modified the milestones: 8.4.0, 8.3.2 Dec 27, 2023
HuidaeCho pushed a commit to HuidaeCho/grass that referenced this pull request Jan 9, 2024
* Update i.evapo.time.html

Improved the explaination of the module with a figure and a reference

* Add example of temporal integration (OSGeo#3280)

Ref: Chemin and Alexandridis 2004 https://ikee.lib.auth.gr/record/270217/files/Chemin-Alexandridis.pdf

---------

Co-authored-by: Edouard Choinière <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual Documentation related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants