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

Remove generate_luc_layer.py #20

Open
mdales opened this issue Jul 20, 2023 · 1 comment
Open

Remove generate_luc_layer.py #20

mdales opened this issue Jul 20, 2023 · 1 comment
Labels
tech debt Not wrong, but could be done better

Comments

@mdales
Copy link
Contributor

mdales commented Jul 20, 2023

Early on we implemented a script to make a LUC layer that covered just the project + boundary area, assembling the data from the individual JRC tiles as necessary.

Since then yirgacheffe has implemented a GroupLayer, which mostly makes this script somewhat redundent. To simplify the code here we should probably switched to using GroupLayer (already in use in newer sections of the pipeline) everywhere and remove this script.
@mdales mdales added the tech debt Not wrong, but could be done better label Jul 27, 2023
@mdales
Copy link
Contributor Author

mdales commented Aug 20, 2024

This is addressed in #113

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tech debt Not wrong, but could be done better
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mdales