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

I-TASSER #1216

Merged
merged 52 commits into from
Jan 6, 2017
Merged

I-TASSER #1216

merged 52 commits into from
Jan 6, 2017

Conversation

valtandor
Copy link
Contributor

New easyconfigs for versions 4.0 and 4.2 of bioinformatics (protein structure) software I-TASSER.

@hpcugentbot
Copy link
Contributor

Automatic reply from Jenkins: Can I test this?

@valtandor
Copy link
Contributor Author

Yes

@hpcugentbot
Copy link
Contributor

Easyconfigs unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/7582/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.


# Can't download from the web site automatically as registration is required.
# The URL is, nevertheless, included here for reference.
#source_urls = ['http://zhanglab.ccmb.med.umich.edu/cgi-bin/download_I-TASSER.cgi?ID=358515010=I-TASSER%(version)s.tar.bz2']
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@valtandor I would stick to mentioning http://zhanglab.ccmb.med.umich.edu/I-TASSER/download/download.php, since this URL contains your personal ID...

@boegel
Copy link
Member

boegel commented May 21, 2016

@valtandor also, Travis reports a missing dep:

EasyBuildError: 'Irresolvable dependencies encountered: Java/1.7.0_51'

Maybe just bump to 1.7.0_80 (latest 1.7.x)?

}

moduleclass = 'bio'
modextravars = { 'IMINTASSERDB' : '/share/db/I-TASSER/%(version)s' }
Copy link
Member

@boegel boegel Sep 9, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@valtandor please move this up, moduleclass should be the last line

@boegel
Copy link
Member

boegel commented Sep 9, 2016

Test report by @boegel
FAILED
Build succeeded for 0 out of 2 (2 easyconfigs in this PR)
node2047.delcatty.os - Linux SL 6.8, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/c627c606cdb8c5adc3484b9de60d308f for a full test report.

@valtandor
Copy link
Contributor Author

@boegel You're right. I had forgotten about site-specific things I had put in. Is this better?

@boegel boegel modified the milestones: 3.1.0, 3.x Jan 6, 2017
@boegel
Copy link
Member

boegel commented Jan 6, 2017

Test report by @boegel
SUCCESS
Build succeeded for 3 out of 3 (2 easyconfigs in this PR)
node2523.golett.os - Linux centos linux 7.2.1511, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/45709e2cb2df5691370328aea0b1be96 for a full test report.

@boegel
Copy link
Member

boegel commented Jan 6, 2017

lgtm

@boegel
Copy link
Member

boegel commented Jan 6, 2017

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2105.delcatty.os - Linux SL 6.8, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/122e240110767e99877230b5497238fa for a full test report.

@boegel
Copy link
Member

boegel commented Jan 6, 2017

Going in, thanks @valtandor!

@boegel boegel merged commit 38c7382 into easybuilders:develop Jan 6, 2017
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