Added percent_full data to all api routes.

[swbae31]

• Added percent_full data to all api routes
• Made capacity() simpler by reusing code
• Edited get_latest_data query (in db.py) so that it is ordered alphabetically by group_name

[evantarrh]

nice :+1:

[jzf2101]

Yay :+1:

[natebrennand]

This logic is replicated in 7 places, maybe we can put it in a function?

capacity_list = calculate_fullness_percentage(fetched_data)

# Add fullness percentage to data
for data, capacity in zip(fetched_data, capacity_list):
    data['percent_full'] = capacity

---

Maybe change calculate_fullness_percentage to add the percent_full value and return the full array? calculate_fullness_percentage --> annotate_fullness_percentage ?

[natebrennand]

it would also be sick if you added a unit test for this so that we don't break it in the future

[jzf2101]

:+1:

[natebrennand]

can you add a test for annotate_fullness_percentage?

[thebrianzeng]

add the test we wrote today! you should actually get the real capacity for the locations though and test more than one location.

On Fri Feb 20 2015 at 6:20:48 PM Nate notifications@github.com wrote:

can you add a test for annotate_fullness_percentage?

— Reply to this email directly or view it on GitHub https://github.com/adicu/density/pull/108#issuecomment-75337690.

[jzf2101]

Yay :+1:

[thebrianzeng]

:+1: