Skip to content

Added some tests for the rust functions #1369

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Open
wants to merge 7 commits into
base: 4.0.0
Choose a base branch
from
Open

Added some tests for the rust functions #1369

wants to merge 7 commits into from

Conversation

gerikson
Copy link
Collaborator

Welcome to deepTools GitHub repository! Please check the following regarding
your pull request :

  • Does the PR contain new feature?
  • Does the PR contain bugfix?
  • Does the PR contain documentation changes?
  • Does the PR contain changes to the galaxy wrapper?
  • [ x] Contains pytests

@gerikson gerikson requested a review from adRn-s January 22, 2025 15:01
@SaimMomin12
Copy link
Collaborator

SaimMomin12 commented Jan 22, 2025 •
edited
Loading

@gerikson Can you change the target of this PR to 4.0.0 instead of master branch?

@WardDeb WardDeb changed the base branch from master to 4.0.0 January 22, 2025 15:28
@pavanvidem
Copy link
Collaborator

@gerikson FYI, @SaimMomin12 and me are creating tests for plotFingerprint, plotPCA and plotProfiler

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@adRn-s adRn-s Awaiting requested review from adRn-s

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gerikson @SaimMomin12 @pavanvidem