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

Test should test all methods #34

Open
Zethson opened this issue Jan 8, 2024 · 0 comments
Open

Test should test all methods #34

Zethson opened this issue Jan 8, 2024 · 0 comments
Assignees

Comments

@Zethson
Copy link
Member

Zethson commented Jan 8, 2024

      - name: Run pipeline with test data
        run: |
          nextflow run ${GITHUB_WORKSPACE} -profile test,conda --souporcell False --generate_anndata False

Currently we're only testing souporcell. Ideally, we should test all methods in a matrix.

ChatGPT will tell you how to write such a matrix @mari-ga . I'd then start including methods one by one to ensure that all of them work with the test profile. You might need to adapt the test profile or write new ones. Consult @wxicu here if necessary.

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

No branches or pull requests

2 participants