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.

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

Documentation examples #391

Merged
merged 12 commits into from
Oct 16, 2023
Merged

Documentation examples #391

merged 12 commits into from
Oct 16, 2023

Conversation

Lilly-May
Copy link
Collaborator

PR Checklist

Description of changes

So far, most public methods in the Documentation did not have examples showing how to use the respective method. Here, we add examples for public methods for the docs to facilitate usage.

Technical details

The following methods do not have a working example yet:

  • pertpy.plot.coda.effects_umap - @Zethson, could you please add / finish the example there?
  • pertpy.plot.dialogue.pairplot (requires the creation of a DialoguePlot object first, which does not match the usual convention for creating plots (I'll open an issue)
  • set_fdr for pertpy.tools.Tasccoda: set_fdr is currently implemented in base_coda but needs to work differently for scCODA and tascCODA (currently it works only for scCODA; I'll open an issue)
  • pertpy.plot.milo.nhood_graph (currently throws a StopIteration error; I'll open an issue)

@github-actions github-actions bot added the enhancement New feature or request label Oct 4, 2023
Copy link
Member

@Zethson Zethson left a comment

Choose a reason for hiding this comment

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

Awesome!

@Zethson Zethson merged commit 67ee210 into main Oct 16, 2023
6 of 7 checks passed
wxicu added a commit that referenced this pull request Oct 16, 2023
…o dev_metadata

* 'dev_metadata' of https://github.com/theislab/pertpy:
  Documentation examples (#391)
  [pre-commit.ci] pre-commit autoupdate (#395)
  Speed up tests by subsampling (#398)
  Installation Apple Silicon (#393)
  Add new distances (#304)
  Fix cinema OT test (#392)
  [pre-commit.ci] pre-commit autoupdate (#390)
  wasserstein distance return type float (#386)
  fix naming of example data in doc examples (#387)
  Add test for test_distances.py Catches error as reported in Issue #385.
  Fix mypy warning for distances Type hint for `groups` reverted, Iterable is too general.
@Lilly-May Lilly-May deleted the feature/docs_examples branch October 21, 2023 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants