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}[GCCcore-10.3.0] minimap2 bumped to 2.20 #13481

Merged

Conversation

sassy-crick
Copy link
Collaborator

(created using eb --new-pr)

@sassy-crick sassy-crick changed the title minimap2 bumped to 2.20 {bio}[GCCcore-10.3.0] minimap2 bumped to 2.20 Jul 22, 2021
@verdurin
Copy link
Member

Test report by @verdurin
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
nuc.lan - Linux Fedora 33, x86_64, Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz, Python 3.9.5
See https://gist.github.com/1447048ff542e10446432f396dc7c21e for a full test report.

@verdurin
Copy link
Member

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@verdurin: Request for testing this PR well received on generoso

PR test command 'EB_PR=13481 EB_ARGS= /apps/slurm/default/bin/sbatch --job-name test_PR_13481 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 17814

Test results coming soon (I hope)...

- notification for comment with ID 885246440 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
generoso-c1-s-4 - Linux centos linux 8.2.2004, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/b8e0de34e245a77b4299b4ddaed7d6fe for a full test report.

'dirs': ['include']
}

sanity_check_commands = ["minimap2 --help"]
Copy link
Member

Choose a reason for hiding this comment

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

@sassy-crick any reason why you've removed part of the check here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No. That was a mistake. Sorry for this, I have corrected that.

@lexming
Copy link
Contributor

lexming commented Jul 29, 2021

Test report by @lexming
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node379.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/807fab4e789744d46844bd1538ca0b0d for a full test report.

@lexming
Copy link
Contributor

lexming commented Jul 29, 2021

Test report by @lexming
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node115.hydra.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz, Python 2.7.5
See https://gist.github.com/37bf983b69a9bae0b68efbb83963233e for a full test report.

@lexming lexming added the update label Jul 29, 2021
@lexming lexming added this to the next release (4.4.2?) milestone Jul 29, 2021
Copy link
Contributor

@lexming lexming left a comment

Choose a reason for hiding this comment

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

LGTM

@lexming
Copy link
Contributor

lexming commented Jul 29, 2021

Going in, thanks @sassy-crick !

@lexming lexming merged commit ab44c99 into easybuilders:develop Jul 29, 2021
@sassy-crick sassy-crick deleted the 20210722151351_new_pr_minimap2220 branch July 30, 2021 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants