Skip to content

Conversation

@jperez999
Copy link
Collaborator

This PR adds a detection step to the tensorflow local operator to check if the data is on cpu, if it is, it is left as a numpy column. We do not change to tf Column because there seem to be issues with casting dtype of eagertensor.

@jperez999 jperez999 added chore Maintenance for the repository ci labels Mar 21, 2023
@jperez999 jperez999 added this to the Merlin 23.03 milestone Mar 21, 2023
@jperez999 jperez999 requested a review from karlhigley March 21, 2023 19:46
@jperez999 jperez999 self-assigned this Mar 21, 2023
@karlhigley karlhigley merged commit 08d4c89 into NVIDIA-Merlin:main Mar 21, 2023
@github-actions
Copy link

Documentation preview

https://nvidia-merlin.github.io/systems/review/pr-300

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Maintenance for the repository ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants