UW-Macrostrat / macrostrat-api

The API for SCIENCE
3 stars 1 forks source link

sections.js: section thickness tabulation #167

Closed cambro closed 8 years ago

cambro commented 8 years ago
"max_thick": _.max(columns[col_id].sections[i].units, function(d) { return d.max_thick; }).max_thick,
"min_thick": _.min(columns[col_id].sections[i].units, function(d) { return d.min_thick; }).min_thick,

The thickness of all units in a section need to be added. Shouldn't this be:

"max_thick": _.sum(columns[col_id].sections[i].units, function(d) { return d.max_thick; }).max_thick,
"min_thick": _.sum(columns[col_id].sections[i].units, function(d) { return d.min_thick; }).min_thick,
cambro commented 8 years ago

And by sum above I mean the javascript equivalent of summing an array.

cambro commented 8 years ago

should also add a field describing the number of units and or including the unit_ids that were used to tabulate the section statistics.