Skip to content

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.

4 participants