Skip to content

Update README.md#12

Closed
JessvLS wants to merge 53 commits intobiof309:masterfrom
JessvLS:master
Closed

Update README.md#12
JessvLS wants to merge 53 commits intobiof309:masterfrom
JessvLS:master

Conversation

@JessvLS
Copy link
Copy Markdown

@JessvLS JessvLS commented Feb 28, 2020

No description provided.

JessvLS added 11 commits April 22, 2020 08:45
Used aligned version of GI.1 - contains N's to accommodate longer noro sequences during mapping
Dropped minimum reads to allow for consensus to be obtained in low coverage regions of the genome
Changed min ID and VP1 cutoff values to match other updated files in package
RAMPART had since been updated by another team and required different packages to run properly. Environment recommendations are taken from the most recent installment of RAMPART projects - SARS-CoV-2 (ncov2019) - https://github.com/artic-network/artic-ncov2019
Every possible amplicon combination was determined for each of the 77 norovirus genotypes
RAMPART instructions updated - still need to establish correct consensus building instructions (the original instructions above do not work)
@leej3
Copy link
Copy Markdown

leej3 commented Apr 25, 2020

To summarize our chat....

Given the complex dependencies of the project, most of your effort will go into simply creating the package and adding some tests.

  • The environment.yml should be moved into the project's root directory
  • The first approach will be to use Conda to package your code.
  • You should modify the circlecic config file to use conda instead of pip to install your project
  • We can help if you struggle with this.

@leej3
Copy link
Copy Markdown

leej3 commented May 9, 2020

superseded by #32

@leej3 leej3 closed this May 9, 2020
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