Skip to content

Rename databricks.py to databricks_notebook.py#59

Merged
sunishsheth2009 merged 1 commit intomlflow:mainfrom
nobutoba:main
Feb 18, 2025
Merged

Rename databricks.py to databricks_notebook.py#59
sunishsheth2009 merged 1 commit intomlflow:mainfrom
nobutoba:main

Conversation

@nobutoba
Copy link
Contributor

Background

Currently, running the driver notebook classification/notebooks/jupyter.ipynb leads to various errors because the databricks namespace mistakenly resolves to classification/notebooks/databricks.py, a module in the same directory.

classification/notebooks/
├── databricks.py
└── jupyter.ipynb

In contrast, the regression template avoids this issue by naming the Databricks driver notebook databricks_notebook.py:

regression/notebooks/
├── databricks_notebook.py
└── jupyter.ipynb

Solution

A simple way to fix this would be to rename classification/notebooks/databricks.py to classification/notebooks/databricks_notebook.py while keeping its content unchanged.

@sunishsheth2009 sunishsheth2009 merged commit c011a78 into mlflow:main Feb 18, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants