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 ci runs (mamba and numpy related) #81

Merged
merged 36 commits into from
Oct 22, 2024
Merged

🚧 fix ci runs (mamba and numpy related) #81

merged 36 commits into from
Oct 22, 2024

Conversation

enryH
Copy link
Member

@enryH enryH commented Oct 7, 2024

  • see if snakemake envs are somehow cached

enryH and others added 26 commits October 7, 2024 10:53
- see if snakemake envs are somehow cached
- might be that I need to create (not sure what change in runner configurations)
- snakemake environment has it's own mamba installation
- auto-activate environment "test"
latest scipy not supported by lifelines
single packages cannot be specified to just ignore the dependencies.
- leave it to njab to install dependencies in a second step.
@enryH
Copy link
Member Author

enryH commented Oct 9, 2024

ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
blis 1.0.1 requires numpy<3.0.0,>=2.0.0, but you have numpy 1.26.4 which is incompatible.
thinc 8.3.2 requires numpy<2.1.0,>=2.0.0; python_version >= "3.9", but you have numpy 1.26.4 which is incompatible.
Successfully installed autograd-1.7.0 autograd-gamma-0.5.0 category-encoders-2.6.4 formulaic-1.0.2 interface-meta-1.3.0 lifelines-0.29.0 llvmlite-0.43.0 mrmr-selection-0.2.8 njab-0.0.8 numba-0.60.0 numpy-1.26.4 polars-1.9.0 pynndescent-0.5.13 umap-learn-0.5.6

lifelines currently still relies on numpy below 2.0

  • other packages use later numpy version. lifelines is installed partly by njab, where it is used to plot km curves. Better to not include it in the default installation then.

Errors were created installing njab separatly without pimms.

@enryH
Copy link
Member Author

enryH commented Oct 17, 2024

  • removed scipy as a direct dependency
  • removed other dependencies which are not used in pkg directly (but might be needed in workflows). njab installs these
  • njab (other repo) now does not always install lifelines
  • fix bugs on Ubuntu with latest package versions

@enryH enryH changed the title 🚧 switch mamba installation 🚧 fix ci runs (mamba and numpy related) Oct 17, 2024
@enryH enryH marked this pull request as ready for review October 21, 2024 10:28
@enryH enryH merged commit ab6b1b2 into main Oct 22, 2024
25 of 26 checks passed
@enryH enryH deleted the fix_ci_runs branch October 22, 2024 18:15
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.

1 participant