Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

migrate base operator to operator and base_runtime to runtime #359

Merged
merged 3 commits into from
Jul 12, 2023

Conversation

jperez999
Copy link
Collaborator

This PR migrates the Base_operator and base_runtime to operator and runtime for both the file names and the class names (where necessary). There are now two basic operator class types, operator and stat operator (which inherits from operator).

@jperez999 jperez999 added the api Changes or tweaks to the Core API label Jul 12, 2023
@jperez999 jperez999 added this to the Merlin 23.07 milestone Jul 12, 2023
@jperez999 jperez999 self-assigned this Jul 12, 2023
@github-actions
Copy link

Documentation preview

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

@jperez999 jperez999 added enhancement New feature or request clean up labels Jul 12, 2023
@jperez999 jperez999 changed the title migrate base operator to operator migrate base operator to operator and base_runtime to runtime Jul 12, 2023
merlin/dag/__init__.py Outdated Show resolved Hide resolved
merlin/dag/operator.py Outdated Show resolved Hide resolved
@jperez999 jperez999 merged commit 2d68307 into NVIDIA-Merlin:main Jul 12, 2023
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Changes or tweaks to the Core API clean up enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants