'pg_seq2seq'
Closed this issue · 7 comments
您好,我尝试了一下 pg_seq2seq,然后我这边报了两个错误。
- 在 super(PGSeq2SeqSolver, self).init('pg_cnn', controller, recorder, counter, **kwargs) 这里多接收了 'pg_cnn'
- 在 seq2seq 中的 SubEnv 少了在调用 super 时少了三个变量 controller, recorder, counter
您这边有什么建议吗?或者怎样修改。
Hello, I'm facing the same problem with @Rorsachach ,and pg_seq2seq`s net has not act method? Could you please give me some suggestions?
Thanks for your feedback. I think the unavailability of pg_seq2seq
results from the last refactorization, and I will attempt to address this problem in my power as soon as possible.
@xuxi2569471573 I did make some simple modifications, but I don't know whether it is what the author intended.
- change
super(PGSeq2SeqSolver, self).init('pg_cnn', controller, recorder, counter, **kwargs)
tosuper(PGSeq2SeqSolver, self).__init__(controller, recorder, counter, **kwargs)
- change
sub_env = SubEnv(p_net, v_net)
tosub_env = self.SubEnv(p_net, v_net, self. Controller, self. Recorder, self. Counter)
. And modify the constructor of SubEnv:
def __init__(self, p_net, v_net, **kwargs):
super(SubEnv, self).__init__(p_net, v_net, **kwargs)
===>
def __init__(self, p_net, v_net, controller, recorder, counter, **kwargs):
super(SubEnv, self).__init__(p_net, v_net, controller, recorder, counter, **kwargs)
Thanks so much! I have made some similar changes, but the important thing is that there is no act method in pg_seq2seq`s net , I stop here. Can you run with these modifications?
@xuxi2569471573 Sorry, I don't know how to write this act either.
Thanks for your feedback and discussion. This bug has been fixed in the latest version (v0.4.0).
Thinks so much!