Skip to content
This repository was archived by the owner on Jul 10, 2025. It is now read-only.

Commit 581d21a

Browse files
alextpmartinwicke
authored andcommitted
clarifications
1 parent cf483b8 commit 581d21a

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

governance/api-reviews.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,9 @@ When adding new ops, look for:
215215

216216
Experimental APIs are APIs which have the word 'experimental' somewhere in their
217217
name; for example `tf.experimental.foo`, or `tf.foo.experimental.Bar`, or
218-
`tf.foo(experimental_bar=True)` or `tf.Foo().experimental_bar()`.
218+
`tf.foo(experimental_bar=True)` or `tf.Foo().experimental_bar()`. We generally
219+
prefer experimental namespaces when possible, so prefer
220+
`tf.foo.experimental.Bar` over `tf.foo.ExperimentalBar`.
219221

220222
Experimental APIs are APIs intended to be added to TensorFlow as-is, but which
221223
we reserve the right to change in backwards-incompatible ways if we have
@@ -246,7 +248,7 @@ Experimental APIs are not a license to break users. This means:
246248

247249
Small changes which are mentioned in relnotes and have obvious fixes might be
248250
made (for example if adding a new argument to a long argument list and we
249-
believe there are no pass-by-position users we might allow the new argument to
251+
believe there are few pass-by-position users we might allow the new argument to
250252
be added to the middle and not the end of the parameter list).
251253

252254
Large backwards-incompatible changes to experimental APIs still require an

0 commit comments

Comments
 (0)