[BUG] Default directories in notebooks don't exist by default

Open
#601 2 comments 0 reactions 1 assignee View on GitHub

@benfred is already working on this.

Since Apr 20, 2021.

Assessment

This issue has not been assessed yet.

Description

bug P1

Describe the bug
Some of the example notebooks have a BASE_DIR variable that points at /raid/data/ml/, which won't exist by default on most systems. Changing it in each notebook gets cumbersome now that examples are split into multiple notebooks

Steps/Code to reproduce bug
Run the MovieLens example notebooks on a system that doesn't have a /raid/data/ml/ directory.

Expected behavior
Once I have the proper dependencies installed, I should be able to select Cell->Run All and have the notebook execute successfully without having to perform manual configuration steps.

Environment details (please complete the following information):

  • Environment location: Bare-Metal
  • Method of NVTabular install: conda

Additional context
N/A

Dominant language
Python
Stars
1.2k
Forks
149
PR merge metrics
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from NVIDIA-Merlin/NVTabular

All issues in NVIDIA-Merlin/NVTabular

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.