{bio,data,lib}[foss/2020b,fosscuda/2020b] AlphaPulldown v0.30.4, cctbx-base v2020.8, Kalign v2.0.4 w/ Python 3.8.6#17774
Conversation
…22.8-foss-2020b.eb, Kalign-2.0.4-GCCcore-11.2.0.eb and patches: AlphaPulldown-0.30.4_fix-deps.patch
| @@ -0,0 +1,34 @@ | |||
| # making AlphaPulldown's dependencies less strict | |||
There was a problem hiding this comment.
some of these are making it a lot less strict. Especially TF from 2.9 to 2.4. Is this known to still work correctly?
There was a problem hiding this comment.
well, we need the 2020b toolchain, as the cctbx-base binary is built for Python 3.8 and there are no alternatives...
when you brought it up, I actually checked the usage and differences between versions.
I've found that all the usages of tensorflow appear to begin with import tensorflow.compat.v1, which suggests that there is no reason to keep the deps this strict.
There was a problem hiding this comment.
I looked into this with Denis, and unfortunately we indeed are forced to stick to Python 3.8 here, unless a lot more effort is made to figure out how to get a from-source install of cctbx-base to work (which doesn't seem trivial).
So we're proceeding under the assumption that the version requirements here are a bit too strict.
If problems emerge when using AlphaPullDown because of older dependency versions, we'll revisit this then
removed unnecessary Kalign
messed up version, removing this one as it uses the wrong version
|
@boegel repaired, thanks for all the feedback. Along the way, I realised I've messed up the version of the original file (even though the file downloaded was correct so apart from easyconfig name, all is good) so I had to delete the original one and replace it with the correctly named one. |
|
@boegelbot please test @ generoso |
|
@boegel: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1541564251 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @boegel |
|
@boegelbot please test @ jsc-zen2 |
|
@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1558868696 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot edit (by @boegel): I've removed |
|
@boegelbot please test @ jsc-zen2 |
|
@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 1570431415 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @boegel |
|
Going in, thanks @deniskristak! |
(created using
eb --new-pr)