search

keras-team / keras-nlp

Modular Natural Language Processing workflows with Keras
Apache License 2.0
762 stars 228 forks source link

Port GPT Text generation guide to multi backend Keras #1147

Open shivance opened 1 year ago

shivance commented 1 year ago

Will work on this parallel to #1146

https://keras.io/examples/generative/text_generation_gpt/

shivance commented 1 year ago

@mattdangerw apologies, not having bandwidth to work on this currently.

monishakant22 commented 1 year ago

@mattdangerw I'm interested in contributing! Could you please assign me this issue?

shivance commented 1 year ago

@monishakant22 you are very welcome to contribute! Please open a PR at keras-team/keras-io for examples We can use this thread to discuss if you need any help!

monishakant22 commented 1 year ago

@shivance can you please help me out how to open a new PR at keras-team . Also can you please share some material to read to get started with open source.

shivance commented 1 year ago

Sure. To make PR

  1. Fork keras-io repo and clone it on your system
  2. Do a pip install -r requirements.txt to set local dev environment. You are all set for development now !
  3. Then you would need to make necessary changes to file for porting tutorial
  4. Commit your changes and push to your fork, github will show you option to open a pull request.

Regarding materials..

  1. You could refer this : https://github.com/keras-team/keras-io/pull/1438 Go to files changed tab and see how that tutorial was ported to Keras Core

Lastly it might seem little overwhelming while starting contributing but with time you would get used to it.

shivance commented 1 year ago

Hi @monishakant22 any update? are you stuck somewhere and need help?

shivance commented 1 year ago

Thanks @monishakant22 , I'm taking over on this issue. Will open a PR soon.

shivance commented 1 year ago

@mattdangerw We are good to go with TF backend here, opened a PR on Keras-io repository. Currently testing with JAX.

github-actions[bot] commented 6 months ago

This issue is stale because it has been open for 180 days with no activity. It will be closed if no further activity occurs. Thank you.