Skip to main content

Browse the archive

Revision ea0bd463cd9d6b6180c8b602fe6629da27d9cf82 authored by Joost van Griethuysen on 30 March 2017, 10:05:44 UTC, committed by Joost van Griethuysen on 30 March 2017, 10:05:44 UTC 
ENH: Simplify specifying a progress reporter
 
As @jcfr showed in #230, the progress reporting can be simplified, requiring the user only to specify one variable that holds a progress reporter.

Update PyRadiomics to this implementation and add examples for both use with `tqdm.tqdm` (which can be used directly) or `click.progressbar` (which requires a simple wrapper class). Examples are provided in both the helloRadiomics script and notebook.

Progress reporting in full-python mode in GLCM and GLSZM now uses a progress reporter that is instantiated with an interator (as opposed to a length) and uses this progress reporter in the loop to iterate over the gray levels. This is encapsulated in a `with` statement to ensure correct closure.
Finally, a dummy progress reporter is provided for the cases where no progressbar is needed (none specified or verbosity level > INFO). In those cases the `_dummyProgressReporter` (defined in `__init__.py`) is used, which accepts the arguments from PyRadiomics and iterates over the gray levels passed in the iterable, but does not report any progress back to the user.
1 parent d4f8837
Raw File
circle.yml 

machine:
  services:
    - docker

dependencies:
  cache_directories:
    - ~/docker
  override:
    - curl -fsSL https://git.io/v2Ifs -o ~/bin/circleci-matrix
    - chmod +x ~/bin/circleci-matrix
    - pip install scikit-ci-addons==0.11.0
    - ci_addons docker load-pull-save dockcross/manylinux-x64
    - ci_addons docker load-pull-save dockcross/manylinux-x86

test:
  override:
     # Workaround https://github.com/pypa/setuptools/pull/971
     - (unlink venv > /dev/null 2>&1 && echo "symlink removed") || echo "no symlink"

     - circleci-matrix:
        parallel: true
back to top