Danijar Hafner

Results 165 comments of Danijar Hafner

@gwelymernans Thanks, that works except for the second example, where the closing parenthesis should not have its own line: ``` python a_very_long_function_name( long_argument_name_1=1, long_argument_name_2=2, long_argument_name_3=3, long_argument_name_4=4) ```

Why did you remove this functionality with your newest commit https://github.com/google/yapf/commit/311c08719451a0cdc3a578b360293ffabb28e0d6?

Yes, this behavior would be very nice, including the breaking before the first argument in the third example but that might be an unrelated concern.

I think this makes only sense for lists. For single-element tuples, the trailing comma is not formatting but syntax.

Thanks! I'll leave this open in case it's helpful for other people. I don't know if updating the env is worth it at this point since OpenAI seems to have...

Running into the same issue here. It's surprising because robosuite uses the same mujoco version as dm_control, but dm_control runs over 3x faster with rendering enabled than robosuite does without...

I also have a gist for a char-rnn in TensorFlow that works very similar to the one described in the book: https://gist.github.com/danijar/c70e7f7811231ce0cecceea76d07ffed