Repository navigation
Commit 024b9db
authored
Wire carbon analytics into API and add impact report endpoint (#124)
* Wire carbon analytics into API and add impact report endpoint
Deforestation runs can now surface carbon loss estimates. Adds an
`enable_carbon` flag to POST /api/predict that, for deforestation
analyses, embeds a `carbon_estimation` block (carbon_tonnes,
hectares_lost, co2_equivalent) computed from the inference pixel count
via the existing analytics.carbon module.
Adds GET /api/reports/{run_id} returning a structured impact report
(hectares_lost, carbon_tonnes, co2_equivalent, confidence_interval,
region_bbox). The report recomputes from the stored pixel count so it
works whether or not carbon was requested at predict time.
Both paths degrade gracefully if the analytics module is unavailable.
Adds tests validating the carbon math against hand-computed IPCC
values and covering both new endpoints.
Closes #14
* Address review: consistent forest_type, labelled CI, strict validation
- /api/reports now reuses the forest_type and region chosen at predict
time (persisted in the carbon_estimation payload) instead of always
assuming tropical_moist, so predict and report agree for a given run.
- Label the confidence interval as tCO2e so it can't be misread as a
band around carbon_tonnes (off by the 44/12 factor).
- Reject an unknown forest_type with 422 instead of silently falling
back to a generic biomass density and returning wrong numbers with 200.
- Add tests for report/predict consistency, the CI unit, and the 422.
* Report from pixel count directly and expose biomass
- Add CarbonEstimator.estimate_from_pixel_count so the impact report no
longer allocates an np.ones(pixels) array just to sum it back; a real
tile can be tens of millions of pixels. estimate_from_mask now delegates
to it, so the two paths stay identical.
- Surface the already-computed biomass_tonnes in the report response.
* Reject unknown region with 422, symmetric with forest_type
An unmatched region (wrong case, trailing space) previously fell back to
a 1.0 adjustment factor and returned a confident but wrong figure. Since
these numbers are meant to be cited, validate region against the known
keys and fail loudly at both /api/predict and /api/reports, mirroring the
existing forest_type guard.1 parent 880bff5 commit 024b9db
3 files changed
Lines changed: 472 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
109 | 109 | | |
110 | 110 | | |
111 | 111 | | |
112 | | - | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
113 | 136 | | |
114 | 137 | | |
115 | 138 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
133 | 133 | | |
134 | 134 | | |
135 | 135 | | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
136 | 148 | | |
137 | 149 | | |
138 | 150 | | |
| |||
185 | 197 | | |
186 | 198 | | |
187 | 199 | | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
| 252 | + | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| 259 | + | |
| 260 | + | |
| 261 | + | |
| 262 | + | |
| 263 | + | |
| 264 | + | |
| 265 | + | |
| 266 | + | |
| 267 | + | |
| 268 | + | |
| 269 | + | |
| 270 | + | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
| 276 | + | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
| 293 | + | |
| 294 | + | |
| 295 | + | |
| 296 | + | |
188 | 297 | | |
189 | 298 | | |
190 | 299 | | |
| |||
669 | 778 | | |
670 | 779 | | |
671 | 780 | | |
| 781 | + | |
| 782 | + | |
| 783 | + | |
| 784 | + | |
| 785 | + | |
| 786 | + | |
| 787 | + | |
| 788 | + | |
| 789 | + | |
| 790 | + | |
| 791 | + | |
| 792 | + | |
| 793 | + | |
| 794 | + | |
| 795 | + | |
| 796 | + | |
| 797 | + | |
| 798 | + | |
| 799 | + | |
| 800 | + | |
| 801 | + | |
| 802 | + | |
| 803 | + | |
| 804 | + | |
| 805 | + | |
| 806 | + | |
| 807 | + | |
| 808 | + | |
| 809 | + | |
| 810 | + | |
| 811 | + | |
| 812 | + | |
| 813 | + | |
| 814 | + | |
| 815 | + | |
| 816 | + | |
| 817 | + | |
| 818 | + | |
| 819 | + | |
| 820 | + | |
| 821 | + | |
| 822 | + | |
| 823 | + | |
| 824 | + | |
| 825 | + | |
| 826 | + | |
| 827 | + | |
| 828 | + | |
| 829 | + | |
| 830 | + | |
| 831 | + | |
| 832 | + | |
| 833 | + | |
| 834 | + | |
| 835 | + | |
| 836 | + | |
| 837 | + | |
| 838 | + | |
| 839 | + | |
| 840 | + | |
| 841 | + | |
| 842 | + | |
| 843 | + | |
| 844 | + | |
| 845 | + | |
| 846 | + | |
| 847 | + | |
| 848 | + | |
| 849 | + | |
| 850 | + | |
| 851 | + | |
| 852 | + | |
672 | 853 | | |
673 | 854 | | |
674 | 855 | | |
| |||
677 | 858 | | |
678 | 859 | | |
679 | 860 | | |
| 861 | + | |
| 862 | + | |
| 863 | + | |
| 864 | + | |
680 | 865 | | |
681 | 866 | | |
682 | 867 | | |
| |||
728 | 913 | | |
729 | 914 | | |
730 | 915 | | |
| 916 | + | |
| 917 | + | |
| 918 | + | |
| 919 | + | |
| 920 | + | |
| 921 | + | |
| 922 | + | |
| 923 | + | |
| 924 | + | |
| 925 | + | |
| 926 | + | |
| 927 | + | |
| 928 | + | |
| 929 | + | |
| 930 | + | |
731 | 931 | | |
732 | 932 | | |
733 | 933 | | |
| |||
0 commit comments