keras-nlp icon indicating copy to clipboard operation
keras-nlp copied to clipboard

Published 20 hours ago • verified publisher icon keras-team

TransformerDecoder should support single inputs

Open chenmoneygithub opened this issue 2 years ago • 2 comments

Currently TransformerDecoder must have both encoder_inputs and decoder_inputs, while for models like GPT2, only decoder_inputs is required. We should make a change to mark encoder_inputs as optional.

chenmoneygithub avatar May 16 '22 20:05 chenmoneygithub

I'll like to take this issue

jessechancy avatar Jun 01 '22 21:06 jessechancy

@jessechancy Thanks!

chenmoneygithub avatar Jun 01 '22 21:06 chenmoneygithub

Is this issue still looking for contributions ? @chenmoneygithub @mattdangerw

shivance avatar Feb 25 '23 18:02 shivance

This is fixed!

mattdangerw avatar Oct 18 '23 21:10 mattdangerw