gpt-2-tensorflow2.0 icon indicating copy to clipboard operation
gpt-2-tensorflow2.0 copied to clipboard

Mismatch in Arguments

Open gvermaredrock opened this issue 5 years ago • 6 comments

In Line 180 of gpt2_model, we define the function as: def distributed_train_step(self,inputs, targets,step,grad_clip=True, clip_value=1.0):

But In line 229, we instantiate it as: train_loss = self.distributed_train_step(inputs, step)

The targets variable / argument is missing.

gvermaredrock avatar Jun 12 '20 10:06 gvermaredrock

Is there a fix for this?

rfebbo avatar Jul 08 '20 16:07 rfebbo

Hi, can you please tell if this will be fixed? Thanks

gvermaredrock avatar Sep 16 '20 18:09 gvermaredrock

Hi,

Sorry for the late reply, I was quite busy in another project, I will fix this and another issue in this repository by end of this week.

Thanks

On Wed, Sep 16, 2020 at 11:53 PM gvermaredrock [email protected] wrote:

Hi, can you please tell if this will be fixed? Thanks

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/akanyaani/gpt-2-tensorflow2.0/issues/5#issuecomment-693579476, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACWLBKBU5YW72DKNXTJ2S73SGD7AXANCNFSM4N4GEXXA .

akanyaani avatar Sep 16 '20 18:09 akanyaani

Thanks Man, really looking forward to it. :)

gvermaredrock avatar Sep 16 '20 18:09 gvermaredrock

Hi @rfebbo, @gvermaredrock , I have fixed the issue, please let me you find another issue.

akanyaani avatar Sep 19 '20 21:09 akanyaani

great, thanks man. please close the bug then as it is misleading

nasheedyasin avatar Oct 22 '20 19:10 nasheedyasin