Skip to content
Snippets Groups Projects
  1. May 17, 2018
    • Neta Zmora's avatar
      Fix system tests failure · a7ed8cad
      Neta Zmora authored
      The latest changes to the logger caused the CI tests to fail,
      because test assumes that the logging.conf file is present in the
      same directory as the sample application script.
      The sample application used cwd() instead, and did not find the
      log configuration file.
      a7ed8cad
  2. May 16, 2018
    • Neta Zmora's avatar
      refactoring: move config_pylogger out of the sample app · 792e9e39
      Neta Zmora authored
      Soon we will be reusing this function in other sample apps, so let's
      move it to app_utils.
      792e9e39
    • Neta Zmora's avatar
      Check if correct version of PyTorch is installed. · ba653d9a
      Neta Zmora authored
      The 'master' branch now uses PyTorch 0.4, which has API changes that
      are not backward compatible with PyTorch 0.3.
      
      After we've upgraded Distiller's internal implementation to be
      compatible with PyTorch 0.4, we've added a check that you are using
      the correct PyTorch version.
      
      Note that we only perform this check in the sample image classifier
      compression application.
      ba653d9a
    • Neta Zmora's avatar
      refactoring: move the message logger setup out of main() · 6e8b0fd6
      Neta Zmora authored
      Eventually we will want to use this code in other sample applications,
      so let's move the logger configuration code to a separate function.
      
      There's a bit of ugly hacking in this current implementation because
      I've added variable members to logging.logger.  These are actaully
      config-once variables that convey the logging directory and filename.
      I did not want to add more names to the global namespace, so I hacked
      a temporary solution in which logging.logger is acts as a conveyor and
      private namespace.  We'll get that cleaned up as we do more refactoring.
      6e8b0fd6
    • Neta Zmora's avatar
      New summary option: print modules names · 6a940466
      Neta Zmora authored
      This is a niche feature, which lets you print the names of the modules
      in a model, from the command-line.
      Non-leaf nodes are excluded from this list.  Other caveats are documented
      in the code.
      6a940466
    • Neta Zmora's avatar
      PNG summary: default to non-parallel graphs · 53b74ca6
      Neta Zmora authored
      Data parallel models may execute faster on multiple GPUs, but rendering
      them creates visually complex and illegible graphs.
      Therefore, when creating models for a PNG summary, we opt to use
      non-parallel models.
      53b74ca6
    • Neta Zmora's avatar
      7ce11aee
  3. May 14, 2018
  4. Apr 24, 2018
Loading