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

Resolve package dependencies for installation #109

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

agitter
Copy link

@agitter agitter commented Mar 25, 2021

Requiring sklearn in setup.py installs the wrong package. The other changes make requirements.txt more consistent with setup.py.

I found it tricky to install this package from requirements.txt and created a compatible conda environment. I have that on a separate branch and could merge it here if you are interested.

@agitter
Copy link
Author

agitter commented Mar 25, 2021

I expanded the scope of this pull request to fix an error I encountered when trying to run the OmicsIntegrator command:

$ OmicsIntegrator
Traceback (most recent call last):
  File "/ua/gitter/miniconda3/envs/oi2/bin/OmicsIntegrator", line 5, in <module>
    from src.__main__ import main
ModuleNotFoundError: No module named 'src'

The edit in 7356b08 fixed this on my system.

@agitter
Copy link
Author

agitter commented Apr 6, 2021

I have a fork that fixes several of the command line-related issues I reported recently and adds basic GitHub Actions testing. I can expand the scope of this pull request to include those updates if you would like them here.

My goal is to prepare an Omics Integrator 2 Docker image that would simplify some of our command line use cases.

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