Skip to content

Add mode=... for data.frame ingestion#360

Merged
eddelbuettel merged 2 commits intomasterfrom
de/sc-13938/extend_df_ingest
Jan 27, 2022
Merged

Add mode=... for data.frame ingestion#360
eddelbuettel merged 2 commits intomasterfrom
de/sc-13938/extend_df_ingest

Conversation

@eddelbuettel
Copy link
Copy Markdown
Contributor

This PR follows the lead ot the Python package and adds a mode argument for array creation from data.frame objects. A unit tests ensures the behaviour of ingest (schema + write), schema_only and append are as expected.

@shortcut-integration
Copy link
Copy Markdown

Copy link
Copy Markdown
Member

@aaronwolen aaronwolen left a comment

Choose a reason for hiding this comment

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

Looks great. Thanks, @eddelbuettel!

@eddelbuettel
Copy link
Copy Markdown
Contributor Author

How about also adding this to the new vignette?

modified   vignettes/data-ingestion-from-sql.md
@@ -105,6 +105,10 @@ storeDataTDB <- function(dat, uri) {
 }

+The mode="append" argument of fromDataFrame can be used to append to an existing array to
+support chunked operation.
+
+

Reading Data Back In

Reading data from TileDB is a very standard operation of opening the URI, possibly specifying the

@eddelbuettel eddelbuettel merged commit d6ed698 into master Jan 27, 2022
@eddelbuettel eddelbuettel deleted the de/sc-13938/extend_df_ingest branch January 27, 2022 17:05
@eddelbuettel eddelbuettel mentioned this pull request Mar 25, 2022
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