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

Less precise event_value and cumulative_probability data types #182

Open
nhoening opened this issue Jun 7, 2024 · 0 comments
Open

Less precise event_value and cumulative_probability data types #182

nhoening opened this issue Jun 7, 2024 · 0 comments
Labels
Database support Dealing with databases

Comments

@nhoening
Copy link
Contributor

nhoening commented Jun 7, 2024

We now use Double Precision data types in the event_value and cumulative_probability columns.

This issue proposes to move to Real types (read more here), so both columns use less space.

The precision loss is moving from 15 to 6 decimal digits. On the plus side, we'd use 8 bytes less per belief, which is more than 10% of all data (or almost 20% if we reduced the space of the belief column).

I researched a bit how to change that value in our ORM (we use Float, see here), but I didn't see yet what change to make here.

I changed the value to Real or Double but the generated migration did not pick this up.

@nhoening nhoening added the Database support Dealing with databases label Jun 7, 2024
@nhoening nhoening changed the title Less precise value and probability data types Less precise event_value and cumulative_probability data types Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Database support Dealing with databases
Projects
None yet
Development

No branches or pull requests

1 participant