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

obsValue datatype is missing #20

Open
aftiqb opened this issue Feb 23, 2012 · 3 comments
Open

obsValue datatype is missing #20

aftiqb opened this issue Feb 23, 2012 · 3 comments
Assignees
Labels

Comments

@aftiqb
Copy link
Member

aftiqb commented Feb 23, 2012

Reported by Soren Roug

We’ve noticed that the sdmx-measure:obsValue doesn’t have a datatype any longer. Is it possible to do some detection of the value so if it is a number, then the datatype is set accordingly? Also, Virtuoso allows scientific notation in xsd:double, but not in xsd:decimal, so we prefer xsd:double.

Further info about this issue can be found in #12

@ghost ghost assigned aftiqb Feb 23, 2012
aftiqb added a commit that referenced this issue Feb 23, 2012
@aftiqb aftiqb closed this as completed Aug 14, 2012
@aftiqb
Copy link
Member Author

aftiqb commented Oct 4, 2012

the bug is still reproducible

@aftiqb aftiqb reopened this Oct 4, 2012
@csarven
Copy link
Member

csarven commented Oct 4, 2012

Just a comment.. why would the data modeling be dependent on how a particular implementation (Virtuoso) works? IMO, xsd:decimal is generally safer to go with without loss of precision.

@aftiqb
Copy link
Member Author

aftiqb commented Oct 4, 2012

Observation values comes in integer, double and string format. Our data modeling is not dependent on Virtuoso or any other implementation. We parse the observations in first iteration to determine the data type of a particular dataset and then assign the appropriate data type to all observation values of that particular data set

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants