Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Remove HugeCTR examples #970

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Remove HugeCTR examples #970

wants to merge 1 commit into from

Conversation

radekosmulski
Copy link
Contributor

resolves #969

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@radekosmulski radekosmulski requested a review from bschifferer May 9, 2023 22:27
@github-actions
Copy link

github-actions bot commented May 9, 2023

Documentation preview

https://nvidia-merlin.github.io/Merlin/review/pr-970

@radekosmulski radekosmulski force-pushed the remove_hugectr_examples branch from e4e5396 to 23c1c44 Compare May 9, 2023 22:51
@radekosmulski radekosmulski added the examples Adding new examples label May 9, 2023
@bschifferer
Copy link
Contributor

Before we merge, we need confirmation from product, that we can remove.

@radekosmulski radekosmulski force-pushed the remove_hugectr_examples branch from 23c1c44 to 5d5ae43 Compare May 11, 2023 00:10
@karlhigley
Copy link
Contributor

@radekosmulski @bschifferer Are we good to go with removing these or still waiting?

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@bschifferer bschifferer Awaiting requested review from bschifferer

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
examples Adding new examples
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Task] Remove HugeCTR examples
3 participants
@radekosmulski @bschifferer @karlhigley