Skip to content

Commit

Permalink
[r] Remove TileDBURI Custom Class (#3689)
Browse files Browse the repository at this point in the history
Remove an unused custom class as part of replacing `TileDBObject` with a
new SOMA base-object class that is not reliant on tiledb-r

[SC-63206](https://app.shortcut.com/tiledb-inc/story/63206)

continues work for #2406
  • Loading branch information
mojaveazure committed Feb 12, 2025
1 parent 9a74797 commit e616a0d
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 167 deletions.
1 change: 0 additions & 1 deletion apis/r/DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ Imports:
utils,
fs,
glue,
urltools,
Rcpp,
data.table,
spdl,
Expand Down
2 changes: 0 additions & 2 deletions apis/r/NAMESPACE
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,5 @@ importFrom(stats,setNames)
importFrom(tibble,as_tibble)
importFrom(tools,R_user_dir)
importFrom(tools,file_path_sans_ext)
importFrom(urltools,url_compose)
importFrom(urltools,url_parse)
importFrom(utils,packageVersion)
useDynLib(tiledbsoma, .registration=TRUE)
5 changes: 3 additions & 2 deletions apis/r/R/TileDBObject.R
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@ TileDBObject <- R6::R6Class(
), call. = FALSE)
}
if (missing(uri)) stop("Must specify a `uri`", call. = FALSE)
private$tiledb_uri <- TileDBURI$new(uri)
# private$tiledb_uri <- TileDBURI$new(uri)
private$tiledb_uri <- uri

# Set platform config
platform_config <- platform_config %||% PlatformConfig$new()
Expand Down Expand Up @@ -195,7 +196,7 @@ TileDBObject <- R6::R6Class(
#' The URI of the TileDB object.
uri = function(value) {
if (missing(value)) {
return(private$tiledb_uri$uri)
return(private$tiledb_uri)
}
stop(sprintf("'%s' is a read-only field.", "uri"), call. = FALSE)
},
Expand Down
109 changes: 0 additions & 109 deletions apis/r/R/TileDBURI.R

This file was deleted.

53 changes: 0 additions & 53 deletions apis/r/tests/testthat/test-04-TileDBURI.R

This file was deleted.

0 comments on commit e616a0d

Please sign in to comment.