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

Feature/Improve batch columnar validation #751

Conversation

figueroa1395
Copy link
Contributor

Continuation of #734: Improves batch validation for columnar data.

Signed-off-by: Santiago Figueroa Manrique <santiago.figueroa.manrique@alliander.com>
@figueroa1395
Copy link
Contributor Author

I might have used git wrongly here. I was supposed just to use #699 as a base to do the fix, but it seems like everything has been copied. @mgovers maybe you have input on this?

@mgovers
Copy link
Member

mgovers commented Oct 1, 2024

I might have used git wrongly here. I was supposed just to use #699 as a base to do the fix, but it seems like everything has been copied. @mgovers maybe you have input on this?

you can try multiple things:

  1. change the base branch of this PR to feature/columnar-dataset-python-extension
  2. rebase
  3. cherry-pick

@figueroa1395 figueroa1395 changed the base branch from main to feature/columnar-dataset-python-extension October 1, 2024 07:16
@figueroa1395 figueroa1395 marked this pull request as ready for review October 1, 2024 07:16
@@ -80,7 +80,7 @@ def test_validate_batch_data_input_error(input_data, batch_data):


def test_validate_batch_data_update_error(input_data, batch_data):
Copy link
Member

Choose a reason for hiding this comment

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

can you add a validation case with columnar input and columnar update dataset?

Base automatically changed from feature/columnar-dataset-python-extension to main October 1, 2024 12:47
@figueroa1395 figueroa1395 changed the base branch from main to feature/columnar-batch-list-support October 7, 2024 14:22
@figueroa1395
Copy link
Contributor Author

Since #760 implemented some necessary changes for this PR and I have made a mess of myself, I will close this PR to open a new relevant one.

@TonyXiang8787
Copy link
Member

@figueroa1395 please delete the branch if it is not needed.

@figueroa1395 figueroa1395 deleted the feature/improve-columnar-validation branch October 8, 2024 07:28
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.

3 participants