Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

TF-Seq2Seq via Docker #258

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Commits on Jun 11, 2017

  1. Configuration menu
    Copy the full SHA
    a31c1fd View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    8053094 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    4448976 View commit details
    Browse the repository at this point in the history
  4. Fix hooks_test.py on Python 3.

    Fixes the following error when running the tests on Python 3.
    
        Traceback (most recent call last):
          File "seq2seq/seq2seq/test/hooks_test.py", line
          55, in test_begin
              self.assertEqual(file_contents.decode(), ...)
              AttributeError: 'str' object has no attribute 'decode'
    darrengarvey committed Jun 11, 2017
    Configuration menu
    Copy the full SHA
    f85d026 View commit details
    Browse the repository at this point in the history
  5. Another go at fixing hooks_test.py.

    distutils.version isn't available on the CI versions, so change the test
    to not rely explicitly on the version of TensorFlow installed.
    darrengarvey committed Jun 11, 2017
    Configuration menu
    Copy the full SHA
    83d592f View commit details
    Browse the repository at this point in the history

Commits on Jun 16, 2017

  1. Configuration menu
    Copy the full SHA
    52c0704 View commit details
    Browse the repository at this point in the history
  2. Fix typo on Docker README

    ReDeiPirati committed Jun 16, 2017
    Configuration menu
    Copy the full SHA
    6b5bc71 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    14e63d4 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    f409d3d View commit details
    Browse the repository at this point in the history
  5. Fix Dockefile missing /n

    ReDeiPirati committed Jun 16, 2017
    Configuration menu
    Copy the full SHA
    8b8f3f2 View commit details
    Browse the repository at this point in the history

Commits on Jun 18, 2017

  1. Tell pylint about the tf contextmanager.

    Use of `tf.name_scope` and `tf.variable_scope` cause pylint errors on TF1.2
    due to pylint not fully understanding the `tf_contextlib.contextmanager`
    decorators.
    darrengarvey committed Jun 18, 2017
    Configuration menu
    Copy the full SHA
    ffb1505 View commit details
    Browse the repository at this point in the history
  2. Work around no-name-in-module pylint errors.

    Following some changes in TF to the `LazyLoader` [1], pylint complains
    about not being able to find some imports under `tf.contrib`. This looks
    like a pylint issue, emitting errors like:
    
        ************* Module seq2seq.encoders.rnn_encoder
        E: 24, 0: No name 'rnn' in module 'LazyLoader' (no-name-in-module)
        ************* Module seq2seq.data.input_pipeline
        E: 32, 0: No name 'slim' in module 'LazyLoader' (no-name-in-module)
    
    [1] tensorflow/tensorflow@95c5d7e
    darrengarvey committed Jun 18, 2017
    Configuration menu
    Copy the full SHA
    0c20ee4 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    514861c View commit details
    Browse the repository at this point in the history
  4. Fix typo

    ReDeiPirati committed Jun 18, 2017
    Configuration menu
    Copy the full SHA
    3aabe47 View commit details
    Browse the repository at this point in the history