search

UW-Macrostrat / macrostrat-api

The API for SCIENCE
3 stars 1 forks source link

column 218 data comes back in /units route, but not /column route #146

Closed jonhusson closed 8 years ago

jonhusson commented 8 years ago

compare

https://macrostrat.org/api/columns?col_id=218

to

https://macrostrat.org/api/units?col_id=218

cambro commented 8 years ago

~~Issue is this condition in the WHERE clause of the query: col_area IS NOT NULL Offshore columns that are based on drill holes have no area. they are points. If this is not acceptable, then check the cols table and makes sure there are no NULLS~~

Issue is that col_id=218 is not in col_areas. Why? This table is of questionable utility these days.

cambro commented 8 years ago

There are 10 such columns with no record in col_areas: select * from cols where id NOT IN (SELECT col_id from col_areas);

jonhusson commented 8 years ago

but in this return, col_id 218 does have an area?

https://macrostrat.org/api/units?col_id=218

v: 2, license: "CC-BY 4.0", data: [ { unit_id: 6494, section_id: 1579, col_id: 218, project_id: 1, col_area: 14554.388, unit_name: "Santa Margarita Fm", strat_name_id: 1823, Mbr: "", Fm: "Santa Margarita", Gp: "", SGp: "", t_age: 10.5265, b_age: 12.17, max_thick: 305, min_thick: 0, outcrop: "subsurface", * pbdb_collections: 0 }

On Mar 28, 2016, at 9:23 AM, Shanan Peters notifications@github.com<mailto:notifications@github.com> wrote:

Issue is this condition in the WHERE clause of the query: col_area IS NOT NULL Offshore columns have no area. they are points. If this is not acceptable, then check the cols table and makes sure there are no NULLS

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHubhttps://github.com/UW-Macrostrat/macrostrat-api/issues/146#issuecomment-202413481

cambro commented 8 years ago

See updates above

jczaplew commented 8 years ago

Should be fixed. Please verify.

jonhusson commented 8 years ago

looks good on this end!

On Mar 28, 2016, at 12:41 PM, John J Czaplewski notifications@github.com<mailto:notifications@github.com> wrote:

Should be fixed. Please verify.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHubhttps://github.com/UW-Macrostrat/macrostrat-api/issues/146#issuecomment-202501969

jczaplew commented 8 years ago

💪