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

gene_nmf: adding the gene NMF novelty libraries #27

Closed
wants to merge 6 commits into from

Conversation

marcdubybroad
Copy link
Collaborator

Needed libraries:

  • numpy
  • scikit-learn (should include numpy)

Possible Docker impacts:

  • will need a python docker image that has support for mathematical libraries
  • I use python:3.8-slim-buster for the base image

Tested with command:

  • pytest --log-cli-level=INFO tests/test_gene_nmf_adapter.py

@maximusunc
Copy link
Collaborator

Thanks @marcdubybroad. Since this doesn't actually plug into the novelty score yet, I'd like @pg427 to add to this PR with the full integration before we merge this in.

@maximusunc
Copy link
Collaborator

Could you also update the requirements.txt for any added dependencies please?

@maximusunc
Copy link
Collaborator

Closing in preference of #30

@maximusunc maximusunc closed this Nov 20, 2024
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