-
Notifications
You must be signed in to change notification settings - Fork 720
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
{bio}[foss/2020b] artic-ncov2019 v2021.06.24 #13852
{bio}[foss/2020b] artic-ncov2019 v2021.06.24 #13852
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs some checks on the used dependencies/extensions to remove those already present in core Python
Besides that: Having trouble understanding how this was generated. For verification purposes: Why and where do those dependencies and extensions come from? Checking the homepage/repo I can't find those so it looks like a more or less random collection of stuff to me which makes it hard to verify
easybuild/easyconfigs/a/artic-ncov2019/artic-ncov2019-2021.06.21-foss-2020b.eb
Outdated
Show resolved
Hide resolved
easybuild/easyconfigs/a/artic-ncov2019/artic-ncov2019-2021.06.21-foss-2020b.eb
Outdated
Show resolved
Hide resolved
{bio}[GCCcore/10.2.0] LSD2 bumped to 2.2, required for artic #13852
…LE3831 {bio}[GCCcore/10.2.0] Muscle bumped to 3.8.31, required for artic #13852
…asyconfigs into 20210830134227_new_pr_artic-ncov201920210621
…or Eigen and TF need to stay as they are though (medaka)!
…asyconfigs into 20210830134227_new_pr_artic-ncov201920210621
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I test-installed fieldbioinformatics
on top of the dependency modules and seemingly the only required packages are:
- clint-0.5.1
- args-0.1.0
- PyVCF-0.6.8
- tqdm-4.62.3
The others are already provided by the Python Module (or others) and can be removed. I'd also suggest to move the above 4 above the fieldbioinformatics package as they are dependencies of it.
easybuild/easyconfigs/a/artic-ncov2019/artic-ncov2019-2021.06.24-foss-2020b.eb
Outdated
Show resolved
Hide resolved
…e name from False to artic
Test report by @Flamefire |
Test report by @Flamefire |
Test report by @Flamefire |
easybuild/easyconfigs/a/artic-ncov2019/artic-ncov2019-2021.06.24-foss-2020b.eb
Show resolved
Hide resolved
@boegelbot please test @ generoso |
@SebastianAchilles: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 926489920 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @boegel |
Test report by @boegel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Test report by @boegel |
Going in, thanks @sassy-crick! |
(created using
eb --new-pr
)