Skip to content

Commit

Permalink
Feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
ebezzi committed Feb 28, 2024
1 parent d126590 commit c221799
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 5 deletions.
10 changes: 8 additions & 2 deletions api/python/cellxgene_census/src/cellxgene_census/_get_anndata.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,8 @@ def get_anndata(
var_value_filter: Optional[str] = None,
var_coords: Optional[SparseDFCoord] = None,
column_names: Optional[soma.AxisColumnNames] = None,
add_obs_embeddings: Optional[Sequence[str]] = (),
add_var_embeddings: Optional[Sequence[str]] = (),
) -> anndata.AnnData:
"""Convenience wrapper around :class:`tiledbsoma.Experiment` query, to build and execute a query,
and return it as an :class:`anndata.AnnData` object.
Expand Down Expand Up @@ -59,10 +61,14 @@ def get_anndata(
Columns to fetch for ``obs`` and ``var`` dataframes.
obsm_layers:
Additional obsm layers to read and return in the ``obsm`` slot.
Use :func:`get_all_available_embeddings` to retrieve available embeddings
for this Census version and organism.
varm_layers:
Additional varm layers to read and return in the ``varm`` slot.
add_obs_embeddings:
Embeddings to be returned as part of the ``obsm`` slot.
Use :func:`get_all_available_embeddings` to retrieve available embeddings
for this Census version and organism.
add_var_embeddings:
Embeddings to be returned as part of the ``varm`` slot.
Use :func:`get_all_available_embeddings` to retrieve available embeddings
for this Census version and organism.
Expand Down
6 changes: 3 additions & 3 deletions tools/census_contrib/src/census_contrib/metadata.py
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ def validate_metadata(args: Arguments, metadata: EmbeddingMetadata) -> Embedding
4. All supplied URLs must resolve
5. Title must have length < 128 characters
6. Description must have length < 2048 characters
7. Name must have length < 24 characters
7. Name must have length < 128 characters
"""
if not metadata.id:
raise ValueError("metadata is missing 'id' (accession)")
Expand All @@ -142,8 +142,8 @@ def validate_metadata(args: Arguments, metadata: EmbeddingMetadata) -> Embedding
"Metadata: description must be string between 1 and {MAX_DESCRIPTION_LENGTH} characters in length",
)

# 7. Name must have length < 24 characters
MAX_NAME_LENGTH = 24
# 7. Name must have length < 128 characters
MAX_NAME_LENGTH = 128
if not metadata.name or len(metadata.name) > MAX_NAME_LENGTH:
raise ValueError(
f"Metadata: name must be string between 1 and {MAX_NAME_LENGTH} characters in length",
Expand Down

0 comments on commit c221799

Please sign in to comment.