search

thinkshout / feeding-texas

Home of the Feeding Texas website. Development by ThinkShout.
feedingtexas.org
5 stars 12 forks source link

QA: Zip code results page - Please check all numbers are pulling from proper CSV columns #107

Closed jcdwyer1 closed 10 years ago

jcdwyer1 commented 10 years ago

There looks to be some data problems iwith some of these elements. I'm happy to help here. data_problem

alexellison commented 10 years ago

There are two issues...one was a bug on our end that I've fixed (via f53f9c5d510c389ff1117e2fcf310b7e87adca4f) and the other, which is still occurring, is there are several zip codes for which totalIncomeEligibleIndividuals is zero. This latter issue seems like one you'll have to fix in zip-data.csv

jcdwyer1 commented 10 years ago

These zeros are unfortunately correct, from a data standpoint. Can we conditionally remove the related text/number/chart in instances where the number is zero or div/0? It's not a common occurrence.

jcdwyer1 commented 10 years ago

I looked more into this. There are 653 instances in zip-data.csv where the number of totalIncomeEligibleIndividuals is less than the number of totalSnapRecipients. These instances result in the first two rows of zip code charts / text getting screwed up like the example below.

Can we conditionally remove these two rows of content any time that totalIncomeEligibleIndividuals < totalSnapRecipients?

If this is too time intensive for today, let me know and I will temporarily redact these instances from the CSV file.

zeroresults2

alexellison commented 10 years ago

@jcdwyer1, I'll take care of conditionally removing today.