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.

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

Enhance plot_lcc_freq_bar Function to Accommodate n Land Cover Types #12

Open
dindiarto opened this issue Jul 14, 2023 · 1 comment
Open
Assignees
Labels
enhancement New feature or request FJ FJ's request/suggestion

Comments

@dindiarto
Copy link
Collaborator

dindiarto commented Jul 14, 2023

The plot_lc_freq function currently supports the comparison of land cover frequencies for two distinct time points (T1 and T2). This issue seeks to enhance the function to accommodate an arbitrary number (n) of time points.

As a guide for the proposed enhancements, we can refer to this implementation, which already accommodates n land cover types.
The enhancements would involve:

  • Modifying the function arguments to accept a variable number of columns for time points. This could be done by replacing column_T1 and column_T2 with a single parameter that accepts a list of column names.

  • Adapting the internal data processing logic to iterate over the list of time point columns. Particularly, this would affect the NA replacement, data reshaping, and max range calculation.

  • Generating separate plots for each time point and concatenating them together. The code that generates the individual bar plots would need to be looped over the list of time point columns.

@dindiarto dindiarto added the enhancement New feature or request label Jul 14, 2023
@dindiarto dindiarto self-assigned this Jul 14, 2023
@dindiarto dindiarto added the FJ FJ's request/suggestion label Jul 14, 2023
@dindiarto
Copy link
Collaborator Author

calc_lc_freq has accomodated n-Land Cover as an input

@dindiarto dindiarto changed the title Enhance plot_lc_freq Function to Accommodate n Land Cover Types Enhance plot_lcc_freq_bar Function to Accommodate n Land Cover Types Jul 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request FJ FJ's request/suggestion
Projects
None yet
Development

No branches or pull requests

1 participant