{bio}[foss/2023a] EVidenceModeler v2.1.0, FASTA v36.3.8i, PASA v2.5.3, pblat v2.5.1#21569
Conversation
…i-GCC-12.3.0.eb, GMAP-GSNAP-2024-09-18-GCC-12.3.0.eb, PASA-2.5.3-foss-2023a.eb, pblat-2.5.1-foss-2023a.eb
|
I had not tested the sanity check command for EVidenceModeler yet. The one that is now included fails with this: The output looks correct, its just the exit code where EasyBuild fails over. Shall I remove |
|
Test report by @laraPPr |
|
@laraPPr I would use You may need to also redirect stderr to stdout, so: |
|
Test report by @laraPPr |
|
@boegelbot please test @ generoso |
|
@laraPPr: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2402054146 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
@laraPPr: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2402132225 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
@laraPPr: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2404287535 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
@laraPPr: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2404367930 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ generoso |
|
@laraPPr: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2404396432 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegel I don't understand why GMAP-GSNAP is failing on Generoso? |
|
@laraPPr: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2414159121 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
| ] | ||
|
|
||
| # Install ParaFly | ||
| postinstallcmds = ["cd %(installdir)s && make"] |
There was a problem hiding this comment.
@laraPPr The Makefile has this:
CFLAGS="-fopenmp" CXXFLAGS="-fopenmp"which should really be
CFLAGS="$CFLAGS -fopenmp" CXXFLAGS="$CXXFLAGS -fopenmp"Without that, ParaFly will be built without any compiler optimizations (default is -O0), which makes it a very slow binary.
Let's add a patch for that?
We should probably report this upstream...
There was a problem hiding this comment.
it fails with this when I try what you suggested configure: error: C++ compiler cannot create executables
There was a problem hiding this comment.
We should also remove the pre-built binary:
| postinstallcmds = ["cd %(installdir)s && make"] | |
| postinstallcmds = ["cd %(installdir)s && rm plugins/ParaFly/bin/ParaFly && make"] |
There was a problem hiding this comment.
EVidenceModeler is no longer maintained so I report it upstream but it probably is not gonna be picked up
|
Test report by @boegel |
| ] | ||
|
|
||
| # Install ParaFly | ||
| postinstallcmds = ["cd %(installdir)s && make"] |
There was a problem hiding this comment.
We should also remove the pre-built binary:
| postinstallcmds = ["cd %(installdir)s && make"] | |
| postinstallcmds = ["cd %(installdir)s && rm plugins/ParaFly/bin/ParaFly && make"] |
|
Test report by @laraPPr |
|
Test report by @laraPPr |
| modextrapaths = { | ||
| 'EVM_HOME': '', | ||
| } | ||
|
|
||
| sanity_check_commands = ["$EVM_HOME/EVidenceModeler -h 2>&1 | grep 'Evidence Modeler'"] |
There was a problem hiding this comment.
@laraPPr Shouldn't we add the top-level installation directory to $PATH?
| modextrapaths = { | |
| 'EVM_HOME': '', | |
| } | |
| sanity_check_commands = ["$EVM_HOME/EVidenceModeler -h 2>&1 | grep 'Evidence Modeler'"] | |
| modextrapaths = { | |
| 'EVM_HOME': '', | |
| 'PATH': ''. | |
| } | |
| sanity_check_commands = ["EVidenceModeler -h 2>&1 | grep 'Evidence Modeler'"] |
There was a problem hiding this comment.
I'm not sure funannotate only uses $EVM_HOME and that is also what their own documentation says. But in a sense their is no harm in adding it. I think.
|
Test report by @laraPPr |
|
@boegelbot please test @ generoso |
|
@laraPPr: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2422195242 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
@boegelbot please test @ jsc-zen3 |
|
@laraPPr: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 2422291518 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Test report by @boegel |
(created using
eb --new-pr)