search

Grist-Data-Desk / land-grab-2

Code and methodology to produce the dataset in Grist's Misplaced Trust investigation
https://grist.org/project/indigenous/land-grant-universities-indigenous-lands-fossil-fuels
Creative Commons Zero v1.0 Universal
14 stars 2 forks source link

parcelCount not exporting as expected #35

Closed clayton-aldern closed 10 months ago

clayton-aldern commented 10 months ago

Hey @mariaparazorose and @parkerziegler, any ideas on what's going on here? I've modified dataset_merge.py (in particular, the dedup_single() function) to calculate a count of duplicate parcels by geometry prior to deduplication and subsequent export. I've also modified constants.py to include the new column. When I run step one, I don't see the new column in the resulting CSVs. Thoughts?

clayton-aldern commented 10 months ago

Also, by the way, this issue should serve as a reminder to remove the 'SKIP_DEDUP': True flags for MN from state_trust_config.py once the above comment is resolved.

parkerziegler commented 10 months ago

@clayton-aldern should we still remove that SKIP_DEDUP: true for MN? I think that'd also allow us to remove ~4 lines in dataset_merge.py since I believe MN is the only state using that flag.

clayton-aldern commented 10 months ago

Oh, yeah; I removed the flags in 24aebd3. I'll clean out the dead lines, too. Thank you!!