Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

t.rast.climatologies: new addon for GRASS 8 #1005

Open
wants to merge 7 commits into
base: grass8
Choose a base branch
from
Open

t.rast.climatologies: new addon for GRASS 8 #1005

wants to merge 7 commits into from

Conversation

lucadelu
Copy link
Contributor

@lucadelu lucadelu commented Dec 30, 2023 •
edited by neteler
Loading

t.rast.climatologies is a module to calculate climatologies, i.e., long term stats, for single days or months in a time series.

@lucadelu lucadelu requested a review from wenzeslaus December 30, 2023 07:15
@neteler neteler changed the title added t.rast.climatologies for GRASS 8 t.rast.climatologies: new addon for GRASS 8 Dec 30, 2023
@neteler neteler added the new addon PR contains a new addon or issue proposes one label Dec 30, 2023
@lucadelu lucadelu force-pushed the t.rast.climatologies.v8 branch from a7111a1 to 87e03ad Compare January 13, 2024 06:16
@ecodiv
Copy link
Contributor

ecodiv commented Jan 15, 2024

Nice, works great. It would be good to also add a weekly granularity (or even user-defined ones). I added to questions/remarks.

@lucadelu
Copy link
Contributor Author

Nice, works great. It would be good to also add a weekly granularity (or even user-defined ones). I added to questions/remarks.

It could be but now I have no time to implement it, I suggest to merge it and later if you like to do that... ;-)

@lucadelu
Copy link
Contributor Author

@wenzeslaus I'm not sure I can write in grass8 branch... I have this

Screenshot from 2024-01-16 06-04-34

@ecodiv
Copy link
Contributor

ecodiv commented Jan 16, 2024 •
edited
Loading

Nice, works great. It would be good to also add a weekly granularity (or even user-defined ones). I added to questions/remarks.

It could be but now I have no time to implement it, I suggest to merge it and later if you like to do that... ;-)

I could certainly give it a try, at some time, bit busy right now :-). What about how the s-flag is described? As I wrote in a previous comment, it seems the description in the manual page doesn't clearly match what the flag is actually doing? Or am I just misreading it?

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

@neteler neteler neteler left review comments

@wenzeslaus wenzeslaus wenzeslaus approved these changes

Assignees
No one assigned
Labels
new addon PR contains a new addon or issue proposes one
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lucadelu @ecodiv @neteler @wenzeslaus