text_classification_rnn.py: Add mandatory padded_shapes argument in padded_batch

[nalepae]

No description provided.

[tfdocsbot]

Preview and run these notebook edits with Google Colab:

• site/en/tutorials/text/text_classification_rnn.ipynb

Notebook diffs available on ReviewNB.com.

[MarkDaoust]

Hi @nalepae,

Thanks for the PR.

I did this in preparation for the 2.2, where the new default behavior is sufficient for these examples.

This notebook has a pip install tf-nightly at the top, so that it runs in Colab.

But I recognize that this is causing some confusion. Maybe I did it a little early. But the 2.2 branch is cut, and will be in rc soon.

If I accept this PR, I'll just have to come back and undo it at release.

So I rather keep it as is.

Thanks again.

[nalepae]

Yes you'r right, I used the wrong tensorflow (NOT the nightly one). But I'm wondering if it is really relevant to use tfnightly in the official documentation. For example I wanted to use padding for a professional use case, where I don't want to use nightly tf. I guess I'm not the only one in this case. Anyway, thank you for your excellent documentation! Le mar. 3 mars 2020 à 14:54, Mark Daoust <notifications@github.com> a écrit :

…

Closed #1486 <#1486>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1486?email_source=notifications&email_token=ABFW7VSETVIJX2G6U7V3EZ3RFUDX5A5CNFSM4LAKTWH2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOXBH7A7I#event-3092246653>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABFW7VSDUDWIIL7PR5HFWR3RFUDX5ANCNFSM4LAKTWHQ> .