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.

Sign up for GitHub

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

Jump to bottom

[r] Fix context-propagation issue #3164 #3213

Closed
johnkerl opened this issue Oct 21, 2024 · 0 comments
Closed

[r] Fix context-propagation issue #3164 #3213

johnkerl opened this issue Oct 21, 2024 · 0 comments
Assignees
@johnkerl @mojaveazure
Labels
bug Something isn't working r-api

Comments

@johnkerl
Copy link
Member

johnkerl commented Oct 21, 2024 •
edited
Loading

chanzuckerberg/cellxgene-census#1261

#3164

[sc-56531]
@johnkerl johnkerl added bug Something isn't working r-api labels Oct 21, 2024
mojaveazure added a commit that referenced this issue Oct 29, 2024
@mojaveazure
[r] Plumb old-style contexts to new libtiledbsoma contexts
eba9d6f 
Allow plumbing tiledb-r contexts `SOMATileDBContext` objects into
libtiledbsoma `soma_context()` objects

Modified SOMA methods:
 - `TileDBObject$new()`: plumb `tiledbsoma_ctx` config options into
   `soma_context()` when provided

[SC-56531](https://app.shortcut.com/tiledb-inc/story/56531)
addresses #3213
@mojaveazure mojaveazure mentioned this issue Oct 29, 2024
Merged
mojaveazure added a commit that referenced this issue Oct 29, 2024
@mojaveazure
[r] Plumb old-style contexts to new libtiledbsoma contexts
f290ff4 
Allow plumbing tiledb-r contexts `SOMATileDBContext` objects into
libtiledbsoma `soma_context()` objects

Modified SOMA methods:
 - `TileDBObject$new()`: plumb `tiledbsoma_ctx` config options into
   `soma_context()` when provided

[SC-56531](https://app.shortcut.com/tiledb-inc/story/56531)
addresses #3213
johnkerl added a commit that referenced this issue Oct 29, 2024
@mojaveazure @johnkerl
[r] Plumb old-style contexts to new libtiledbsoma contexts (#3252)
467af01 
* [r] Plumb old-style contexts to new libtiledbsoma contexts
Allow plumbing tiledb-r contexts `SOMATileDBContext` objects into
libtiledbsoma `soma_context()` objects

Modified SOMA methods:
 - `TileDBObject$new()`: plumb `tiledbsoma_ctx` config options into
   `soma_context()` when provided

[SC-56531](https://app.shortcut.com/tiledb-inc/story/56531)
addresses #3213

* add no-sign-request to S3 test cases

* Update changelog
Bump develop version

---------

Co-authored-by: John Kerl <[email protected]>
@johnkerl johnkerl closed this as completed Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johnkerl johnkerl

@mojaveazure mojaveazure

Labels
bug Something isn't working r-api
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@johnkerl @mojaveazure