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

Iterator refactor #14

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

Conversation

sordonia
Copy link
Owner

  • Refactored SS_iterator, removed useless arguments.
  • Refactored data_iterator.py: i moved the Iterator class outside of the method get_data_iterator and created a method get_test_iterator to get a separate test iterator.
  • I also included a pair of TODO that should be addressed in a future.

julianser and others added 15 commits May 4, 2015 21:08
…ndition is not satisfied, all models will fail to run.
…ate flag to set wether or not the L2 pooling is used on the utterance encoders output.
…utterance encoder(s) and utterance decoder. Also factorized UtteranceDecoder to make it easier to add future inputs to the it. In particular, I renamed \'hs\' to \'inp\' in UtteranceDecoder.
…antic similarities between targets and samples using SEMILAR: A Semantic Similarity Toolkit. The setup should be directly transferable to clusters for parallel fast evaluation (less than a few hours per model).
…coders and direct-connection-between-encoder-and-decoder.
…merging Alessandro's corrections and refactoring of the beam and stochastic search.
…ed 'index' in data_iterator, so that we can now retrieve the index of each triple. This will be used to pair it with semantic information.
Added DCGM + modified search to remove prev_h
julianser added a commit to julianser/hed-dlg that referenced this pull request May 21, 2015
…onal baclwards compatability and exception handling code.
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.

2 participants