chanzuckerberg / cellxgene-census

CZ CELLxGENE Discover Census
https://chanzuckerberg.github.io/cellxgene-census/
MIT License
84 stars 20 forks source link

switch our r-universe to track GitHub Releases #688

Closed mlin closed 1 year ago

mlin commented 1 year ago

get help from central infra to merge: https://github.com/chanzuckerberg/chanzuckerberg.r-universe.dev/pull/7

mlin commented 1 year ago

Fri evening update:

https://chanzuckerberg.r-universe.dev/builds

The cellxgene.census build is okay (except for Windows which isn't expected to work). The tiledbsoma macOS build is not because the required version of tiledb is not published through the same r-universe (nor CRAN).

image

Potential solutions, in my guessed order of preference:

  1. Update our user installation instructions to install tiledbsoma (and transitively tiledb) from tiledb-inc.r-universe.dev instead of ours. Like the current tiledb instruction, this would be temporary until the necessary tiledb version is up on CRAN (at which point the tiledbsoma build on chanzuckerberg.r-universe.dev should work again).
  2. Add tiledb to chanzuckerberg.r-universe.dev as well as tiledbsoma, by copying this stanza from tiledb-inc's packages.json.
  3. Beg for a tiledbsoma 1.4.1 with a DESCRIPTION updated to allow the current CRAN version of tiledb. (This would potentially leave users exposed to the query hanging bug, if they don't follow our instruction to explicitly install the newest version beforehand)
pablo-gar commented 1 year ago

Complete in https://github.com/chanzuckerberg/chanzuckerberg.r-universe.dev/pull/8 and https://github.com/chanzuckerberg/chanzuckerberg.r-universe.dev/pull/7