-
Notifications
You must be signed in to change notification settings - Fork 14
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
Update DESDM dtypes #147
Comments
Current DESDM convention can be found in eautils/dtypes.py |
Alex,
Originally (previous DB admin) these columns were set to a number type to allow for better indexing... however, this switched within the Y3 release as the more prevalent use cases showed that these columns were being used in arithmetic expressions where having the values cast as a binary floats would allow queries to execute faster.
Robert
…________________________________
From: Alex Drlica-Wagner [[email protected]]
Sent: Wednesday, June 06, 2018 9:40 PM
To: mgckind/easyaccess
Cc: Gruendl, Robert A; Mention
Subject: [mgckind/easyaccess] Update DESDM dtypes (#147)
It appears that the DESDM database type convention has changed for Y3. The MAG and MAGERR columns have changed from NUMBER to be BINARY_FLOAT. @mgckind<https://github.com/mgckind> @rgruendl<https://github.com/rgruendl> can you confirm that this is the current (and future) convention?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#147>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AOWegwNr8jI4jhjpqjMi9EnjPgvCxu1iks5t6JKegaJpZM4Udpf0>.
|
Thanks @rgruendl. Can I take the format of Y3 to be the new "standard"? If so, I will update all dtypes to match that. |
yes, you can take it as the new standard
…________________________________
From: Alex Drlica-Wagner [[email protected]]
Sent: Thursday, June 07, 2018 9:49 AM
To: mgckind/easyaccess
Cc: Gruendl, Robert A; Mention
Subject: Re: [mgckind/easyaccess] Update DESDM dtypes (#147)
Thanks @rgruendl<https://github.com/rgruendl>. Can I take the format of Y3 to be the new "standard"? If so, I will update all dtypes to match that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#147 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AOWeg7PzaP9eby72Dx2_J5QxmLIy34Bkks5t6T2OgaJpZM4Udpf0>.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It appears that the DESDM database type convention has changed for Y3. The
MAG
andMAGERR
columns have changed fromNUMBER
to beBINARY_FLOAT
. @mgckind @rgruendl can you confirm that this is the current (and future) convention?The text was updated successfully, but these errors were encountered: