Skip to content

Commit

Permalink
Big refactor (#124)
Browse files Browse the repository at this point in the history 
* massive revision inspired by soup: algo folder works

* porting rl commands, WIP

* various

* git subrepo push [email protected]:openai/codegen.git --branch=refactor codegen

subrepo:
  subdir:   "codegen"
  merged:   "aa27e069"
upstream:
  origin:   "[email protected]:openai/codegen.git"
  branch:   "refactor"
  commit:   "aa27e069"
git-subrepo:
  version:  "0.4.0"
  origin:   "[email protected]:ingydotnet/git-subrepo.git"
  commit:   "74339e8"

* various

* rewrite RL stuff in new framework

* fix almost everything

* woohoo tests pass

* more tests

* reformatting

* fixes

* write tests for embeddings

* re-remove cg2

* pylint

* minor

* move smooth_helpers import; seems to cause nondeterministic failure in parallel pytest
  • Loading branch information
@joschu @pzhokhov
joschu authored and pzhokhov committed Oct 3, 2018
1 parent e820b86 commit 4cf7dc9
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions baselines/common/distributions.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,9 @@ def param_placeholder(self, prepend_shape, name=None):
def sample_placeholder(self, prepend_shape, name=None):
return tf.placeholder(dtype=self.sample_dtype(), shape=prepend_shape+self.sample_shape(), name=name)

def __eq__(self, other):
return (type(self) == type(other)) and (self.__dict__ == other.__dict__)

class CategoricalPdType(PdType):
def __init__(self, ncat):
self.ncat = ncat
Expand Down

2 comments on commit 4cf7dc9

@wookayin
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see commit messages and the diff do not match. Why is that?

@pzhokhov
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in the internal collection of repos that commit had a lot of changes. Almost all of the changes are unrelated to baselines, but there was a small necessary change (you see it in the diff) in baselines as well.

Please sign in to comment.