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.

Sign up for GitHub

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

Jump to bottom

Remove sample A|B|C columns in layer table #165

Open
jomey opened this issue Oct 23, 2024 · 0 comments
Open

Remove sample A|B|C columns in layer table #165

jomey opened this issue Oct 23, 2024 · 0 comments
Labels
DB Any requests related to changes/updates of the DB table structure or definitions enhancement New feature or request

Comments

@jomey
Copy link
Member

jomey commented Oct 23, 2024

A group discussion with @aaarendt, @micah-prime, and @micahjohnson150 concluded that we are better off not storing three columns with one layer row, but have three entries for the same top and bottom height to indicate a x-sampled value for that layer.
@jomey jomey added enhancement New feature or request DB Any requests related to changes/updates of the DB table structure or definitions labels Oct 23, 2024
jomey added a commit that referenced this issue Oct 23, 2024
@jomey
Tables - LayerData - Drop sample and flag columns
8cf4bc9 
This gets rid of columns sample_a|b|c and the flags column in favor of
storing three values for identical top and bottom height. The flags
column information will move to the comments. Also adding not-null and
index constraints for better data integrity.

Fixes #165
@jomey jomey mentioned this issue Oct 23, 2024
Merged
jomey added a commit that referenced this issue Oct 23, 2024
@jomey
Tables - LayerData - Drop sample and flag columns
a8e9ca6 
This gets rid of columns sample_a|b|c and the flags column in favor of
storing three values for identical top and bottom height. The flags
column information will move to the comments. Also adding not-null and
index constraints for better data integrity.

Fixes #165
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DB Any requests related to changes/updates of the DB table structure or definitions enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jomey