Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix: #362
doyBySeasons
I slightly tuned the
doyByYear
method by adding 2 new parameters:seasonStart
andseasonEnd
. These 2 parameters are set as integer-like value and mark the start and the end of the season.I decided to go for integer because that simplify downstream code and Python GEE API and pure python datetime can both return such value:
plot_doy_by_seasons
Here the trick was to manage the ticks and thankfully
matplotlib
was super easy to tune. I recover the max and min value from the x value of the graph and simply restric what I display between these 2 edges.side effect
As I modified the way doy graph are drawn it has slightly modified the figure for existing tests. I'm satisfied with the adjustment (+5, -5 days of margin in the graph) so I simply reloaded the images.
documentation
I upgraded the documentation usage section and added few executed code for the image_collection file that is anyway to big to manage in one PR.
demonstration
with correctly prepared data you can get the following: