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

{bio} [goolf-1.7.20] QIIME-1.9.1 and dependencies (WIP) #3069

Closed
wants to merge 46 commits into from

Conversation

pescobar
Copy link
Member

@pescobar pescobar commented May 18, 2016

This works for me but it won't work out of the box for others. The BLAST easyconfig has a hardcode path to my blastdb files. Many source tarballs need to be manually downloaded.

I have tested all the easyconfigs so it supports offline installation (no internet access required to download python libs)

This is heavily based on a previous PR from @boegel #1057 and adapted to the requirements of latest QIIME 1.9.1

@boegel boegel added this to the v2.x milestone May 18, 2016
@boegel
Copy link
Member

boegel commented May 18, 2016

@pescobar If you intend to get this merged, it'll have to be split up into smaller PRs, this is going to be impossible to merge.

@pescobar
Copy link
Member Author

@boegel not urgent for me to get it merged "soon". I added the PR mainly to have it as reference and in case it's useful for anyone else but in any case I will try to get it merged in coming weeks/months unless you have a better one for QIIME

@verdurin
Copy link
Member

We've had a request for this, so I'd be happy to help.

@fgeorgatos
Copy link
Contributor

Splitting the PR has the downside of very fragmented merge attempts with
all the extra work that it implies... It has happened before a few times !

On Wednesday, 24 August 2016, Adam Huffman notifications@github.com wrote:

We've had a request for this, so I'd be happy to help.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#3069 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdgQDFQn3vBR7XJZ_fNJkkmhirq0Hkrks5qi9pogaJpZM4IhcTP
.

echo "sysadmin know better bash than english"|sed s/min/mins/
| sed 's/better bash/bash better/' # signal detected in a CERN forum

@pescobar
Copy link
Member Author

pescobar commented Dec 6, 2016

This PR requires an easyconfig for matplotlib/1.4.3. QIIME doesn't work with higher matplotlib versions


source_urls = ['http://www.cpan.org/src/%(version_major)s.0']
sources = [SOURCELOWER_TAR_GZ]

# workaround for one extension check
# be sure you don't build this in NFS folder or you will also get some error
prebuildopts = 'export PERL_LWP_SSL_VERIFY_HOSTNAME=0 && unset INSTALL_BASE && '
Copy link
Member

Choose a reason for hiding this comment

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

@pescobar do you remember why you had to include this exactly?

Copy link
Member Author

Choose a reason for hiding this comment

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

@verdurin
Copy link
Member

verdurin commented Dec 6, 2016

@pescobar - would you accept PRs to provide this for foss and intel 2016b?

@pescobar
Copy link
Member Author

pescobar commented Dec 6, 2016

@verdurin I cannot commit to spend too much time with this PR this week. I am overloaded of work.

I can try to work a little bit on this but I cannot commit sorry

@pescobar
Copy link
Member Author

pescobar commented Dec 6, 2016

@verdurin btw, if your need to get QIIME installed quick check this one #3868

@boegel
Copy link
Member

boegel commented Dec 6, 2016

@verdurin I'm working on QIIME with intel/2016b in bits & pieces, PRs working towards it coming up

Once that is in place, doing the same with foss/2016b should be trivial.

@verdurin Are you up for reviewing my incoming PRs?

@RvDijk
Copy link
Contributor

RvDijk commented Jan 10, 2017

@boegel ping on the intel and foss versions?

@boegel
Copy link
Member

boegel commented Jan 10, 2017

@RvDijk little progress I'm afraid... This is certainly still on my to do list, but it's buried below other stuff that needs to be taken care of first.

I doubt I'll have a lot of time for this, until mid-February.

Are you willing to help out? Maybe we can join efforts on this together with @verdurin?

@verdurin
Copy link
Member

@boegel - just received a reminder from a user about this, so happy to collaborate, yes.

@RvDijk
Copy link
Contributor

RvDijk commented Jan 11, 2017

@boegel I'm also happy to help.

@boegel
Copy link
Member

boegel commented Jan 11, 2017

@verdurin, @RvDijk I opened a WIP PR with the QIIME easyconfig (see #3993), but most of the dependencies are still missing... I have some of them as WIP (e.g. biom-format, emperor, scikit-bio), I'll look into those ASAP.

Any help here is appreciated, each of the dependencies should be checked whether they're still a dep, which version requirements they have, etc.
See https://github.com/biocore/qiime/blob/master/setup.py

@migueldiascosta
Copy link
Member

@pescobar, this PR is being closed for the following reason(s): no activity for > 6 months, obsoleted by more recent PRs.
Please don't hesitate to reopen this PR or add a comment if you feel this contribution is still relevant.
For more information on our policy w.r.t. closing PRs, see https://easybuild.readthedocs.io/en/latest/Contributing.html#why-a-pull-request-may-be-closed-by-a-maintainer

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.

6 participants