search

NVIDIA-Merlin / core

Core Utilities for NVIDIA Merlin
Apache License 2.0
19 stars 14 forks source link

migrate metadata operators to core #344

Closed jperez999 closed 1 year ago

jperez999 commented 1 year ago

This PR migrates all the add metadata operators to core. This will allow them to be used in other parts of merlin, like systems. These are general operators that manipulate schemas. They are not required to be located in the nvtabular repo. By moving these operators to core, we remove the nvtabular import requirement for these ops.

github-actions[bot] commented 1 year ago

Documentation preview

https://nvidia-merlin.github.io/core/review/pr-344

oliverholworthy commented 1 year ago

Should we add these to merlin/dag/ops/__init__.py like the other ops we have in Core? so that they can be imported from merlin.dag.ops instead of requiring an import of this add_metadata module?