fix keras compatibility bug in factorized_top_k

[getumen]

The bug is summarized by the following issue comment.
#712 (comment)

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[mat10599]

@maciejkula

[ghost]

+1 for this PR getting merged in. It was a blocker for me until I manually applied this fix.

[takoyaro]

Can we please get this merged?

[rmminusrslash]

I have the same issue. most active contributor seems to be @maciejkula, can you please tell us how to get this merged and released?

Edit: This change is not sufficient to fix the bug. We also need to change how the counter is accessed, this part in the Streamingclass throws an exception

      """Enumerates rows in each batch using a total element counter."""

      starting_counter = self._counter.read_value()

Who is running CI/CD for the tutorial code? https://www.tensorflow.org/recommenders/examples/quickstart
It throws an exception due to the bug in this PR, so it appears the tensorflow recommender library might not be maintained if even the quickstart does not work

[czroth]

@ZhaoyueCheng, it looks like you are the only active commiter to main in the last 6 mo. Can you give us a hand?

[ZhaoyueCheng]

I don't have permission to merge PRs actually, but since this repository is mostly running with legacy keras, would using TF_USE_LEGACY_KERAS=1 work as a workaround with older keras version? (also suggested in #712 (comment))

[Az-r-ow]

I don't have permission to merge PRs actually, but since this repository is mostly running with legacy keras, would using TF_USE_LEGACY_KERAS=1 work as a workaround with older keras version? (also suggested in #712 (comment))

Passing TF_USE_LEGACY_KERAS=1 is not doing it for me, using the legacy version of keras is making my notebook kernel crash without any logs :

[error] Disposing session as kernel process died ExitCode: undefined, Reason: 

It should be related to the fact that I’m using some of keras latest features, however at this stage of my project it’s frustrating to go back and adjust the code in question.

[y-71]

@rmminusrslash that issue is independant, it has been reported at #722 and there is this patch #723 linked to it

[keivanipchihagh]

+1
I applied this change and fixed the issue described in #712.

[stampixel]

CAN WE GET THIS MERGED????

[ghost]

CAN WE GET THIS MERGED????

The Merge has been pending for quite a long time. In the meantime, you could use alternatives as mentioned here

[piyushwadhwani]

we need this fix please

[mihaimaruseac]

#765 which merged should have this fixed. Sadly, there's no autoclose of this and duplicates.