cgq-qgc / pyhelp

A Python library for the assessment of spatially distributed groundwater recharge and hydrological components with HELP
MIT License
17 stars 5 forks source link

PR: Add options to specify a period in water budget plots #87

Closed jnsebgosselin closed 2 years ago

jnsebgosselin commented 2 years ago

Add the option to specify a year_from and year_to between which monthly and yearly values are calculated and plotted.

The methods affected by the changes made in this PR are:

output_help.plot_area_monthly_avg(year_from=2003, year_to=2007)
output_help.plot_area_yearly_avg(year_from=2003, year_to=2007)
output_help.plot_area_yearly_series(year_from=2004, year_to=2009)

Also, various improvements were made to the graphs produced by the above methods.

Figure_1

Figure_2

Figure_3

pep8speaks commented 2 years ago

Hello @jnsebgosselin! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 98:80: E501 line too long (80 > 79 characters) Line 100:80: E501 line too long (80 > 79 characters) Line 102:80: E501 line too long (81 > 79 characters) Line 103:80: E501 line too long (81 > 79 characters) Line 124:80: E501 line too long (82 > 79 characters) Line 125:80: E501 line too long (81 > 79 characters) Line 126:80: E501 line too long (82 > 79 characters) Line 127:80: E501 line too long (81 > 79 characters) Line 128:80: E501 line too long (83 > 79 characters) Line 129:80: E501 line too long (83 > 79 characters)

Comment last updated at 2022-04-27 15:05:08 UTC
jnsebgosselin commented 2 years ago

@FHuchet Qu'est-ce que tu en penses au niveau des années, est-ce que c'est suffisant où il serait mieux d'ajouter comme tu proposerais Année(s) considérée(s) pour le bilan : ?

image

jnsebgosselin commented 2 years ago

@FHuchet Voici une version alternative où l'information en lien avec la plage des années est plutôt présentée comme une note en bas du graphique au lieu du titre de l'axe des x.

image

FHuchet commented 2 years ago

J'aime bien la deuxième version avec le "années considérées" en note.

FHuchet commented 2 years ago

La forme au pluriel ("années considérées") se met-elle automatiquement à jour quand il y a plus d'une année ou devons-nous la modifier manuellement dans le script ? Les deux me vont. :)

jnsebgosselin commented 2 years ago

J'aime bien la deuxième version avec le "années considérées" en note.

Parfait, je vais y aller comme cela alors.

La forme au pluriel ("années considérées" se met-elle automatiquement à jour quand il y a plus d'une année ou devons-nous la modifier manuellement dans le script ? Les deux me vont. :)

C'est automatique.