-
Notifications
You must be signed in to change notification settings - Fork 32
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
Area Estimation for Multiple RoIs and Time Periods #411
Comments
@adebowaledaniel requesting your review for this issue. |
@hannah-rae and @ivanzvonkov requesting your review. |
Well you've asked for a review, and a different view is what you'll get! Anyway, the issues you mention resemble the issues I had when I started running intercomparison on new countries/regions. However, in the end what made my life easier was a template notebook. I would duplicate the template notebook each time I needed an intercomparison and make minor adjustments to address issues that came up. I think a template area estimation notebook that you can duplicate for each ROI and year will address the issues you bring up. It'll also double as a record of the results. To throw a haymaker in here, if one of the biggest time sucks is actually dealing with downloading and organizing the GeoTiffs I would consider exploring area estimation in Google Earth Engine. It might be easier to bring the code to the data, rather than bring the data to the code each time. I had started exploring this and have some code but it's not fully there. |
@ivanzvonkov I agree that in the case of the intercomparison, a template notebook was useful because we usually want to run it for one country/year at a time for a given project and there are very few details that need to be set for each one. Also, the main result that we want to get from that is the code snippet and the plot of all scores at the end. For annual area estimation, we actually have been using a template notebook already. The issue is that even with the template notebook, there are a lot of things that need to be set for each run and copying the various results and logging the steps along the way is a pain and prone to errors. I personally think that a configurable/modular approach that Gedeon is suggesting would be super valuable. I think we could also look at GEE in the future, but for now the python version is still what we use most and having the configurable version of that would be helpful, even if we decide to move to GEE in the future. |
Issue Description
The current area estimation notebook functions well for individual cases, but when dealing with multiple regions of interest (RoIs) and different time periods, it becomes prone to errors. Each time you want to estimate the area for another RoI or period, you need to rerun the entire notebook. Additionally, the results are only printed, requiring manual copying to store them, which increases the chance of errors and slows down the estimation process.
Suggestion
Develop a separate reusable script or notebook that uses a configuration file to handle multiple RoIs and time periods. This setup will streamline the estimation process by allowing batch processing of RoIs, automating the recording of results, and improving overall efficiency and accuracy.
TODOs
The text was updated successfully, but these errors were encountered: