Updated function signatures to comply with new tensorflow requirements

siyerp commented 2 months ago

Description

I tried to run the sarsec notebook, but ran into issues because the function calls didn't comply with updated tensorflow requirements. I fixed the function calls so that they comply with updated requirements.

Related Issues

If you try running the sarsec notebook with a recent version of tensorflow, the training throws an error because tensorflow modules can only take non-tensor arguments if they are keyword arguments, so the 'training' argument, which is a boolean, throws an error unless it is a keyword argument.

References

Checklist:

[x ] I have followed the contribution guidelines and code style for this project.
[ ] I have added tests covering my contributions.
[x ] I have updated the documentation accordingly.
[x ] I have signed the commits, e.g. git commit -s -m "your commit message".
[x ] This PR is being made to staging branch AND NOT TO main branch.

siyerp commented 2 months ago

@miguelgfierro I'd be interested in collaborating.

miguelgfierro commented 2 months ago

@siyerp would you be able to join the next Monday meeting: Every Monday, we have a recurrent meeting, the time is 4.30-5pm (Central European Time): https://teams.microsoft.com/l/meetup-join/19%3acf53f267b15340de8867aa869ced4ed1%40thre[…]2c%22Oid%22%3a%22c798428d-eb18-456e-aed9-c2ecae3a8510%22%7d

recommenders-team / recommenders