chanzuckerberg / cellxgene-census

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

Update analytics with the new contrib embeddings method #1038

Closed ebezzi closed 6 months ago

ebezzi commented 8 months ago

Update the Census analytics script to account for the new distribution method for collaboration embeddings, when it gets released.

Proposed solution:

  1. Both hosted and collaboration embeddings should now use the same script as the current hosted embeddings (i.e., read from a specific S3 location outside of the Census)
  2. Use a rule to differentiate hosted vs collaboration: if the accession key contains contrib, then it's hosted, if it's czi, then it's collaboration
  3. Note that rule 3 might change in the future and we may have to switch to a mapping.

The engineering team will let @ainfeld know when the change is rolled out.