biocore / biocore/mmvec

Simplify delivery and interfacing

Open
#111 5 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
133
Forks
54
PR merge metrics
No merged PRs in 30d

Description

Hi guys,
congratulations for the Nature methods paper on this.

I have a couple of suggestions:

- support running the tool from more standard input files rather than .biom (.csv, .tab. etc.), that do not need special libraries to handle.
- specify versions of all python libraries needed. Currently, this does not install, as you appear to be using an old version of tensorflow. Even better, provide a Docker image.
- minimize dependencies as far as possible. Even better, provide a Docker image.

When other labs will try to publish similar methods, they will be expected to compare the results to your method and these will make peoples life easier.

Kind regards,
Martin

Contributor guide

No contributing guide indexed for this repository

Research direction

No files, tests, or entry points are named. Start by locating the input handling and Python dependency configuration, then determine the scope of supporting CSV/tabular inputs, specifying library versions, reducing dependencies, and optionally providing a Docker image; done means these usability and installation concerns are addressed.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, python, tensorflow
Domain
devops, machine-learning
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.