Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adds predenominated to ensure raw measures are returned #325

Merged
merged 2 commits into from Dec 9, 2017

Conversation

andy-esch
Copy link
Contributor

@andy-esch andy-esch commented Dec 9, 2017

This PR adds non-denominated measures (aka raw ones) to the metadata response.

ToDo

  • metadata returns non-denominated ("predenominated") values as well as derived quantities
  • add pgquote function to turn strings to single-quoted strings and Nones to nulls
  • ensure that the metadata table is a set (no duplicates) --> it seems that duplicates are not returned from metadata, but this could easily be fixed by adding a distinct to the obs_getmeta call.

closes #324

@coveralls
Copy link

coveralls commented Dec 9, 2017

Coverage Status

Coverage remained the same at 97.309% when pulling 0997ead on iss324-data-discovery-raw-data into d2e9017 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 97.315% when pulling fd9c3c3 on iss324-data-discovery-raw-data into d2e9017 on master.

3 similar comments
@coveralls
Copy link

coveralls commented Dec 9, 2017

Coverage Status

Coverage increased (+0.006%) to 97.315% when pulling fd9c3c3 on iss324-data-discovery-raw-data into d2e9017 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 97.315% when pulling fd9c3c3 on iss324-data-discovery-raw-data into d2e9017 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 97.315% when pulling fd9c3c3 on iss324-data-discovery-raw-data into d2e9017 on master.

@andy-esch andy-esch merged commit 538bcca into master Dec 9, 2017
@andy-esch andy-esch deleted the iss324-data-discovery-raw-data branch December 9, 2017 05:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

not able to get raw data from data_discovery
2 participants