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

Do not update local jgo test unless -u or -U option specified #33

Merged
merged 1 commit into from
Jan 16, 2019

Conversation

hanslovsky
Copy link
Member

Fixes #27

For comparison, run this script:

#/usr/bin/env sh

if [ -z "${CONDA_PREFIX}" ]; then
    export PATH=$HOME/.local/bin:$PATH
fi

echo "Time with update"
time jgo -u org.janelia.saalfeldlab:paintera:0.8.2 --version 2>/dev/null
echo
echo
echo "Time without update"
time jgo org.janelia.saalfeldlab:paintera:0.8.2 --version 2>/dev/null

On latest conda-forge release:

Time with update

real    0m1.956s
user    0m5.883s
sys     0m0.240s


Time without update

real    0m1.912s
user    0m5.728s
sys     0m0.275s

On this PR branch:

Time with update

real    0m1.846s
user    0m5.401s
sys     0m0.270s


Time without update

real    0m0.369s
user    0m0.550s
sys     0m0.070s

@hanslovsky hanslovsky merged commit a5f78d4 into master Jan 16, 2019
@hanslovsky hanslovsky deleted the pyjgo-cache-dir-update branch February 27, 2019 20:25
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.

1 participant