nasaharvest / crop-mask

End-to-end workflow for generating high resolution cropland maps
Apache License 2.0
95 stars 28 forks source link

Add notebook for CEO meta analysis #276

Closed bhyeh closed 8 months ago

bhyeh commented 1 year ago

Notebook for generating simple meta insights to CEO labeling project.

Remaining items and thoughts:

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

bhyeh commented 1 year ago

2/20/23 meeting notes

  1. For crop land map and crop type map - gather validation and test set via CEO and two labeler sets a. For doing just crop land/type mapping - disagreements are 'thrown out' and ignored inputs to model b. For additionally doing area estimation - disagreements are resolved/forced into agreement because 'throwing out' or ignoring these points affect the sample size $\rightarrow$ downstream effect for map accuracy assessment (e.g., width of confidence interval).
  2. Reasons for meta analysis on disagreement points a. For just crop land/type mapping - looking at patterns of disagreements help to build a systematic resolvement or protocol instead of just ignoring #275 b. For area estimation - help identify ambiguous points to be used towards 'counterfactual' or alternative accuracy assessment

Notes:

  1. Most typical is crop land/type map project $\rightarrow$ where result of CEO labeling is two csv files from labeler sets. There is no 'final' label known (b/c disagreement points are just ignored, not forced into agreement, as opposed to area estimate label project).
ivanzvonkov commented 1 year ago

FYI pre-commit is now working and there's a couple issues it highlights (see pre-commit.ci "Details" link)

image
ivanzvonkov commented 1 year ago

@bhyeh let me know when this is ready for a second look!

ivanzvonkov commented 1 year ago

flake8 has some issues with the consensus_utils file, you can see what they are by clicking on the pre-commit.ci "Details"

ivanzvonkov commented 1 year ago

This is looking pretty good, any thoughts about some of the notebook comments? I am happy to merge soon and iterate if need be.

bhyeh commented 1 year ago

This is looking pretty good, any thoughts about some of the notebook comments? I am happy to merge soon and iterate if need be.

Thanks! Yes I do plan on making further changes on the notebook - waiting for some feedback from Hannah and will then address your comments there as well.

ivanzvonkov commented 1 year ago

@bhyeh can this be merged in the near term (after pre-commit issues are addressed)?

bhyeh commented 1 year ago

@bhyeh can this be merged in the near term (after pre-commit issues are addressed)?

I think I need to touch base w/ @hannah-rae for her thoughts. I'll bring this up again with our 1:1 this Monday.