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

Clear dataset columns when creating a new dataset #95

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

dvince2
Copy link
Contributor

@dvince2 dvince2 commented Feb 28, 2024

If a user is saving a modified dataset there is a likelyhood that columns and/or data have changed, so the HMI-Server needs a clue that the columns information should be reset.

If a user is saving a modified dataset there is a likelyhood that columns and/or data have changed, so the HMI-Server needs a clue that the columns information should be reset.
@dvince2 dvince2 requested a review from mattprintz February 28, 2024 18:02
Copy link
Contributor

@mattprintz mattprintz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me

@dvince2 dvince2 merged commit 81417b3 into main Feb 28, 2024
2 checks passed
@dvince2 dvince2 deleted the feat/reset_column_data_on_new_dataset branch February 28, 2024 18:45
dvince2 added a commit to DARPA-ASKEM/terarium that referenced this pull request Feb 28, 2024
This is actually the fix for incorrect column information after a beaker-driven transform too. There is a corresponding change on beaker that blows away column data (DARPA-ASKEM/askem-beaker#95) since columns and/or data will have changed in modified datasets. Then on a GET call we repopulate the column data because its empty.  This will also cover any situation where a CSV/dataset was some how uploaded and the column data was empty too.
In addition, this centralizes some code around getting CSV files and updating headers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants