Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[r] Remove last private$.tiledb_array and self$object from R classes [WIP] #3129

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

johnkerl
Copy link
Member

@johnkerl johnkerl commented Oct 4, 2024

Context: #3059 / [sc-55685]

As detailed in #3059 this is blocked by #3124.

@johnkerl johnkerl changed the title [r] Remove last tiledb_array and self$object from R classes [WIP] [r] Remove last private$.tiledb_array and self$object from R classes [WIP] Oct 4, 2024
arr <- tiledb::tiledb_array(uri)
tsch <- tiledb::schema(arr)

# XXX CHANGEME
Copy link
Member Author

@johnkerl johnkerl Oct 4, 2024

Choose a reason for hiding this comment

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

The massive line-counts of occurrences of tiledb:: on test-SOMASparseNDArray.R, test-SOMADenseNDArray.R, and test-SOMADataFrame.R are due mainly to using tiledb-r to introspect on compression filters etc. that we written using tiledbsoma-r and platform_config but for which we lack a way to read back and verify in tiledbsoma-r -- or tiledbsoma-py for that matter -- cc @nguyenv for thoughts ...

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant