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

Deletes: adding examples. #3437

Merged
merged 2 commits into from
Aug 12, 2022
Merged

Deletes: adding examples. #3437

merged 2 commits into from
Aug 12, 2022

Conversation

KiterLuc
Copy link
Contributor

@KiterLuc KiterLuc commented Aug 9, 2022

This adds examples for deletes.


TYPE: IMPROVEMENT
DESC: Deletes: adding examples.

This adds examples for deletes.

---
TYPE: IMPROVEMENT
DESC: Deletes: adding examples.
@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #20032: Deletes: write example..

Copy link
Member

@ypatia ypatia left a comment

Choose a reason for hiding this comment

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

LGTM, few suggestions on improving readability for the user that would be consulting this example.

// Prepare the vector that will hold the result.
// We take an upper bound on the result size, as we do not
// know a priori how big it is (since the array is sparse)
std::vector<int> data(3);
Copy link
Member

Choose a reason for hiding this comment

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

wouldn't the upper bound be 16 in that case for a 4x4 array?
Or you mean we don't know if any cells were deleted from our initial data that had a size of 3?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the second case in your or question. This was copied from another example.

examples/cpp_api/deletes.cc Show resolved Hide resolved
*
* The MIT License
*
* @copyright Copyright (c) 2018-2022 TileDB, Inc.
Copy link
Member

Choose a reason for hiding this comment

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

NIT: only 2022

examples/c_api/deletes.c Outdated Show resolved Hide resolved
@KiterLuc KiterLuc merged commit f6a9e67 into dev Aug 12, 2022
@KiterLuc KiterLuc deleted the lr/deletes-examples/ch20032 branch August 12, 2022 12:56
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