mstenta / farm_pcsc

EXPERIMENTAL - DO NOT USE
GNU General Public License v2.0
0 stars 1 forks source link

Supplemental workbook validation list updates #30

Closed paul121 closed 2 months ago

paul121 commented 4 months ago

"No significant changes, just validation list updates & verification check"

mstenta commented 2 months ago

I wish they provided diffs for the validation list changes. Adding those lists was very tedious to begin with, and it seems that the only way to update them is to essentially go through the same process again (thereby making our own diffs).

One example I've already come across (while moving "Field commodity volume unit" from "Field summary" to "Farm commodity volume unit" in "Farm summary"), which illustrates changes that would be easy to miss:

+          'Bunches',
           'Bushels',
           'Carcass weight pounds',
+          'Cartons',
+          'Eggs',
+          'Flats',
           'Gallons',
           'Head',
           'Linear feet',
-          'Lightweight pounds',
+          'Liveweight pounds',
+          'Pints',
           'Pounds',
           'Tons',
+          'Trees',
           'Other (specify)',

It's mostly additions, but one appears to be a typo correction. This would be easy to miss unless we're re-entering all these lists again from scratch.

Part of me wonders if we should just leave the lists as-is and rely on the new "Validation Summary" sheet to catch any potential issues - which we can then fix as they are discovered. That would be less work upfront, but might mean more work fixing things later. I'm leaning towards re-copying all lists from scratch just to ensure we're matching (and crossing fingers that USDA doesn't release a new version with more undocumented changes).

mstenta commented 2 months ago

Did a quick pass through validation lists. Documenting the changes that I found (and fixed) here:

Producer enrollment

(none)

Field enrollment

Farm summary

(see previous comment)

Field summary

I have not checked the "Commodity type" list yet. The v2 workbook does not appear to include it, but seems to have a link to it, which is not working for me (says "Invalid range" when I click it in LibreOffice Calc). A list is also provided in the data dictionary PDF, but it's more difficult to copy from there. So this is still TBD.

Note that also I did not check the State/County lists because they are gigantic, and I can't imagine they have changed in any meaningful way that would affect us. (Famous last words! :laughing:)

mstenta commented 2 months ago

I also have not checked all of the decimal value validations. I would imagine those haven't changed drastically, and if they have we will catch them in the DPR validation checks I think.

mstenta commented 2 months ago

I have not checked the "Commodity type" list yet.

Turns out these are in the Validation sheet all the way at the end. I updated our list. There were only two additions.

I think we can consider this done.