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

code review pull request #32

Open
wants to merge 208 commits into
base: codereview
Choose a base branch
from
Open

code review pull request #32

wants to merge 208 commits into from

Conversation

koadman
Copy link
Owner

@koadman koadman commented Mar 4, 2015

trying out this approach for complete code review

cerebis and others added 30 commits February 10, 2014 16:54
Replicons are now selected from within cells only, not across community
Single replicon cells are handled correctly.
eliminate overlapping fragment errors
set min distance to next restriction site to 3
Merge branch 'master' of https://github.com/koadman/proxigenomics

Conflicts:
	simulation/hic_simulator/src/simForward.py
…on of Nestly.

After discussions with the developers of Nestly and a review of other workflow tools (namely Swift) it was decided to stay with Nestly. The SCons
wrapper enables defining sources and targets (dependencies) as well as moving within the hierarchy of the parameter sweep (nest). Moving within the
tree has not been fully exploited as yet.

A few pieces of short-running python code are not being executed on the queue. (fix me)

Currently only recall, precision and f1 are being calculated.

Matt.
cerebis and others added 30 commits June 1, 2015 12:54
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.

2 participants