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

Remove Optic temp files by default #2212

Merged
merged 3 commits into from
Apr 5, 2019
Merged

Conversation

charleygros
Copy link
Member

@charleygros charleygros commented Apr 5, 2019

This PR aims at removing Optic temp files by default.

The files are not removed if verbose >= 2.

Fixes #2209.

@charleygros charleygros changed the title Remove optic temp by default Remove Optic temp files by default Apr 5, 2019
@charleygros charleygros self-assigned this Apr 5, 2019
@jcohenadad jcohenadad merged commit 7ed69ac into master Apr 5, 2019
@jcohenadad jcohenadad deleted the cg/2209-optic-rm-temp branch April 5, 2019 13:46
@jcohenadad jcohenadad added the enhancement category: improves performance/results of an existing feature label Apr 5, 2019
@jcohenadad jcohenadad added this to the v4.0.0 milestone Apr 5, 2019
jcohenadad pushed a commit that referenced this pull request Dec 18, 2019
* optic: remove temporary files by default

* centerline/core: add verbose to detect_centerline

* sct_propseg: add verbose to detect_centerline


Former-commit-id: 7ed69ac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement category: improves performance/results of an existing feature sct_get_centerline context:
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants