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

cmaps #42

Merged
merged 11 commits into from
Feb 13, 2024
Merged

cmaps #42

merged 11 commits into from
Feb 13, 2024

Conversation

WardDeb
Copy link
Member

@WardDeb WardDeb commented Feb 1, 2024

Initially these changes are to deal with the matplotlib - deeptools mismatch when deeptools < 3.5.3 and mpl >= 3.8, but some additional things came up. In brief:

  • float calls on series will be deprecated and need iloc
  • yaml load is deprecated, switched to safe_load
  • loosened the version requirements, but updated those for deeptools & python
  • installation instructions updated to use pip instead of setup.py (works without any change for now it seems)
  • implemented a minimal runner that runs the tests in the repo, changed the relative paths to be the repo root to accomodate the runner

@WardDeb WardDeb requested a review from LeilyR February 1, 2024 09:49
@WardDeb WardDeb merged commit dc5211c into develop Feb 13, 2024
1 check passed
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.

2 participants