Skip to content

ODM-12845 Add cell ratio docs#502

Merged
eeliane merged 9 commits intodevelopfrom
feature/ODM-12845-cell-ratio
Nov 17, 2025
Merged

ODM-12845 Add cell ratio docs#502
eeliane merged 9 commits intodevelopfrom
feature/ODM-12845-cell-ratio

Conversation

@olepark-gs
Copy link
Contributor

@olepark-gs olepark-gs commented Nov 6, 2025

image

@olepark-gs olepark-gs requested review from a team as code owners November 6, 2025 15:15
@olepark-gs olepark-gs changed the base branch from develop to feature/ODM-12846-gene-summary-analytics November 6, 2025 15:15
@olepark-gs olepark-gs changed the title ODM-12845 Add cell ratio decs ODM-12845 Add cell ratio docs Nov 10, 2025
Copy link
Contributor

@m-smazhevsky m-smazhevsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Please add a screenshot of a page
  • analysis -> analytics
  • Consider renaming of CellRatioRequest to CRRequest, CellRatioResponse to CRResponse to shorten python sdk calls (I did it with DifferentialExpression)

…ODM-12845-cell-ratio

# Conflicts:
#	openapi/v1/integrationCurator.yaml
#	openapi/v1/integrationUser.yaml
Base automatically changed from feature/ODM-12846-gene-summary-analytics to develop November 13, 2025 12:32
Copy link
Contributor

@m-smazhevsky m-smazhevsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should explicitly add descriptions of

  • countSelected
  • countAvailable
  • ratio
    to integrationUser/Curator yamls, see how it's done for other 2 endpoints

* order of endpoints
* added [BETA] mark and a disclaimer
* fixed address for user cell-ratio
@m-smazhevsky m-smazhevsky self-requested a review November 14, 2025 11:56
@eeliane eeliane enabled auto-merge November 17, 2025 15:58
@eeliane eeliane merged commit ff3150f into develop Nov 17, 2025
2 checks passed
@eeliane eeliane deleted the feature/ODM-12845-cell-ratio branch November 17, 2025 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants