TensorLayer icon indicating copy to clipboard operation
TensorLayer copied to clipboard

transformer model based on Tensorlayer

Open ArnoldLIULJ opened this issue 6 years ago • 15 comments

Checklist

  • [x] I've tested that my changes are compatible with the latest version of Tensorflow.
  • [x] I've read the Contribution Guidelines
  • [x] I've updated the documentation if necessary.

Motivation and Context

Description

ArnoldLIULJ avatar Jul 22 '19 09:07 ArnoldLIULJ

Documentations haven't been done

ArnoldLIULJ avatar Jul 22 '19 09:07 ArnoldLIULJ

The dependency on Keras should be removed gradually.

tf.keras.optimizers -> tf.optimiziers tf.keras.initializers -> tl.initializers

JingqingZ avatar Jul 22 '19 10:07 JingqingZ

ready to merge?

zsdonghao avatar Sep 01 '19 02:09 zsdonghao

Add attention visualisation util Add attention visualisation test

ArnoldLIULJ avatar Sep 01 '19 10:09 ArnoldLIULJ

ready to merge?

not yet

ArnoldLIULJ avatar Sep 01 '19 10:09 ArnoldLIULJ

Add documentation Add attention-weights visualisation and pass unit-testing READY TO MERGE

ArnoldLIULJ avatar Sep 02 '19 09:09 ArnoldLIULJ

Hi, could you provide an example code in the examples folder? and update changelog.md ? thanks

zsdonghao avatar Sep 02 '19 13:09 zsdonghao

There are some naming issues with this PR. Please don't merge it for now.

luomai avatar Sep 02 '19 14:09 luomai

@ArnoldLIULJ any update?

luomai avatar Sep 12 '19 10:09 luomai

@ArnoldLIULJ any update?

was on vocation and would be working on a simplified tutorial today

ArnoldLIULJ avatar Sep 13 '19 10:09 ArnoldLIULJ

Add examples in example/translation_task/tutorial_transformer

ArnoldLIULJ avatar Sep 13 '19 22:09 ArnoldLIULJ

Hi, could you provide an example code in the examples folder? and update changelog.md ? thanks

done

ArnoldLIULJ avatar Sep 13 '19 22:09 ArnoldLIULJ

Hi the RST format is not correct in many function, please check~

zsdonghao avatar Sep 14 '19 08:09 zsdonghao

please check

ArnoldLIULJ avatar Sep 18 '19 13:09 ArnoldLIULJ

I think this one can be merged after the travis pass~.

zsdonghao avatar Sep 23 '19 02:09 zsdonghao