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

fix custom labels in plotCoverage #1175

Closed
wants to merge 4 commits into from

Conversation

kciy
Copy link

@kciy kciy commented Nov 16, 2022

  • 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?

Copy link
Member

@bgruening bgruening left a comment

Choose a reason for hiding this comment

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

Thanks @kciy

@kciy
Copy link
Author

kciy commented Nov 30, 2022

@bgruening any updates?

adRn-s and others added 2 commits February 3, 2023 15:09
* Fix label options

* add pool

* trigger test

* new azure syntax?

* run also on PR

* fix deeptools#1159

* fixes deeptools#1172

* Addressing deeptools#1181

* Bigwig average (deeptools#1169)

* Fixes deeptools#1159

* Revert "dendogram of plotCorrelation now matches each cell correctly"

* add bigwigAverage

* linting

* remove forgotten prints

* make bigwigAverage executable

* fix galaxy tests

Co-authored-by: A.s <62971995+adRn-s@users.noreply.github.com>
Co-authored-by: LeilyR <25766687+LeilyR@users.noreply.github.com>

* Addresses deeptools#1183

* Fix galaxy tests (deeptools#1186)

* upload artifact

* allow 1 or 2 lines diff for bowtie2 program

* change github action to get artifacts

* fix plotPCA

* try to fix old samtools installed

* add forgotten channels

---------

Co-authored-by: Cristóbal Gallardo <gallardo@informatik.uni-freiburg.de>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: LeilyR <25766687+LeilyR@users.noreply.github.com>
Co-authored-by: Lucille Delisle <lucille.delisle@epfl.ch>
Co-authored-by: adRn-s <salatino@omics.dev>
@adRn-s adRn-s changed the base branch from master to develop February 8, 2023 14:06
@adRn-s
Copy link
Collaborator

adRn-s commented Feb 8, 2023

Tests should pass now.

The master branch is* awfully outdated, there are plans to release a new deepTools version with all the year-long commit history that was accumulated onto develop branch.

*edit: was, before #1185

PS. The master branch should be protected probably.

@bgruening bgruening deleted the branch deeptools:develop February 8, 2023 20:50
@bgruening bgruening closed this Feb 8, 2023
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