-
Notifications
You must be signed in to change notification settings - Fork 15
Environment and directory structure updates #8
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This is a temporary setup and could change later. For now, each user (class User read here: https://docs.djangoproject.com/en/3.0/ref/contrib/auth/) has projects (class Project) and each project has jobs (class Job). Each class creates a data table and is mapped to the related table using foreign keys. Please read more here: https://docs.djangoproject.com/en/3.0/ref/models/fields/#foreignkey
This reverts commit cbb42d4.
This reverts commit 4ec643d.
This reverts commit cc2d595.
This reverts commit 4d7c87b.
This reverts commit 0270bf1.
This reverts commit c2c5202.
This reverts commit aa1dbc1.
…ctory with 78Mbp simulated community. updated config files. py2.7 bhsne for kmers in its own script to run py2.7 version. removed shebang specifically specifying python3 to avoid cryptic errors where user defined python env is not selected when run. Added .gitignore to ncbi dir under databases to keep empty directory. Post-processing are in their own directory under validation.
I'm going to have to finish my review of the refactored code first before I can assess this. Is that OK? I just would feel better understanding the code first. |
Yes, no problem. Since I have a fork of the repository, I can operate on my own repo until the changes are merged. |
…reflect checkpointing. Renamed MAG class to Mag to follow python conventions. Added prodigal parsing to lca.py. Reflected in majority_vote.py. removed superfluous attributes for DiamondResult object. Updated metagenome.config with checkpoints.tsv file.
…and hmmer.py). By default will pass in ORFs retrieved from Mag object.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
Something isn't working
documentation
Improvements or additions to documentation
enhancement
New feature or request
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is partially to display the git workflow via forks, pull requests, reviews and merges. You do not need to do a comprehensive review here. These edits are minor and should work out some of the kinks I have encountered while testing on the Boreal datasets as well as on my Mac
Updates
tests/metagenome.config
lca.py
to display thousand delimiter for ORFs search description.bhsne.py
under binning directory to provide user capability to run python2.7 BH-tSNE (Note: conda env setup is withinbhsne.py
script.hmmer.py
main script where input and output args were the same. Updated argparse parameters accordingly.kmers.py
kmers.py
args with default--n-components=2
and--do-pca
as well as--pca-dimensions
are now available in the main call to kmers.Example usage of scripts populating datasets directory