Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Apr 27, 2022

follow-up for #3995, since just testing import of easybuild is way too generic

@boegel boegel requested a review from bartoldeman April 27, 2022 11:55
Copy link
Contributor

@bartoldeman bartoldeman left a comment

Choose a reason for hiding this comment

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

LGTM

@bartoldeman
Copy link
Contributor

$ time python -c "import easybuild.framework"

real    0m0.030s
user    0m0.018s
sys     0m0.012s
$ time python -c "import easybuild"

real    0m0.029s
user    0m0.022s
sys     0m0.007s
$ time python -c "import easybuild.main"

real    0m30.081s
user    0m0.654s
sys     0m0.279s
$ time python -c "import easybuild.main"

real    0m0.605s
user    0m0.517s
sys     0m0.143s

it's cheap enough, so let's go for this!

@bartoldeman bartoldeman merged commit f275169 into easybuilders:develop Apr 27, 2022
@boegel boegel deleted the eb_import_check branch April 27, 2022 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants