search

JoshuaPeddle / Visualizing-Technology-Use-by-Age-and-Region

Comp3100. Team 32. Joshua Peddle & Ethan Mc Donald.
0 stars 1 forks source link

build_database.js issue with Usage object construction #22

Closed ethantmc closed 2 years ago

ethantmc commented 2 years ago

I've updated the tests with a Comprehensive Test for Usages, but it looks like the wrong data is being assigned to ageGroup, Income, and uom. Geo, serviceType, and value all pass. Is there some kind of processing going on I'm not aware of, like removing all whitespace?

Originally posted by @ethantmc in https://github.com/JoshuaPeddle/Visualizing-Technology-Use-by-Age-and-Region/issues/14#issuecomment-1069101074

JoshuaPeddle commented 2 years ago

There is a PR that should address some of these errors.

JoshuaPeddle commented 2 years ago

value was being stored as a string. ageGroup and income had some issues with extra quotes being saved. I'll have to take a look at uom.

JoshuaPeddle commented 2 years ago

image

Usage collection seems to look okay now.

JoshuaPeddle commented 2 years ago

image

This was what it was like pre-PR. I think that was causing issues with validation.

JoshuaPeddle commented 2 years ago

image image

I think this is the issue with uom. I think it should be called unit as this is simulating a Usage object.