We were testing against the release-23.02 branch of HugeCTR/SOK, but we should be testing against the main branch. Since we've made the horovod tests optional for merge, breaking changes from SOK should not block Models pipeline.
This PR also fixes the ImportError in the unit test for next item prediction notebook introduced in #1045 due to unavailability of merlin.systems in the test environment by removing the import to pytest.importorskip.
github-actions[bot]
commented
1 year ago
release-23.02branch of HugeCTR/SOK, but we should be testing against the main branch. Since we've made the horovod tests optional for merge, breaking changes from SOK should not block Models pipeline.merlin.systemsin the test environment by removing the import topytest.importorskip.