Addons: Fork Keras Layer Test

Created on 21 May 2019  路  5Comments  路  Source: tensorflow/addons

Per https://github.com/tensorflow/tensorflow/issues/28601 it has been recommended that we fork and expose keras.layer_test for this project and others to utilize.

Any thoughts where we should put test utilities? I think for this one it makes sense somewhere under tfa.layers, but we might want the central tfa.utils location for all test utilities?

good first issue help wanted layers
Source
picture seanpmorgan

Most helpful comment

I would go for tfa.utils.

picture WindQAQ on 22 May 2019
👍2

All 5 comments

My initial thought is tfa.utils. What's your opinion, Sean, Tzu-Wei? cc @WindQAQ

facaiy picture facaiy on 22 May 2019
👍1

I would go for tfa.utils.

WindQAQ picture WindQAQ on 22 May 2019
👍2

Forking the fully functional layer test from keras is quite a challenge. It has calls to other private functions.

Squadrick picture Squadrick on 26 Jan 2020

tfa.utils lgtm

AakashKumarNain picture AakashKumarNain on 26 Jan 2020

Is this still valid?

bhack picture bhack on 22 Jul 2020
Was this page helpful?
0 / 5 - 0 ratings

Related issues

seanpmorgan picture seanpmorgan  路  4Comments
seanpmorgan picture seanpmorgan  路  3Comments
facaiy picture facaiy  路  3Comments
SoufianeDataFan picture SoufianeDataFan  路  4Comments
jimthompson5802 picture jimthompson5802  路  3Comments