Skip to content

Add multiple notebooks #150

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.

Sign up for GitHub

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Add multiple notebooks #150

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d98b23b
fix(notebook): suggestions after test usage
bharathts07 Oct 10, 2025
d3f5068
fix(notebook): suggestions after ML sync
bharathts07 Oct 13, 2025
1b90461
Merge branch 'singlestore-labs:master' into master
bharathts07 Oct 30, 2025
374a312
add(image) : udf
bharathts07 Oct 30, 2025
faff541
update flake 8 pre-commit version
bharathts07 Oct 30, 2025
02f1343
add NITs and suggestions
bharathts07 Oct 31, 2025
b33f4f3
update ml classify to improve reruns
bharathts07 Oct 31, 2025
17ff6b0
Merge branch 'master' into add-multiple-notebooks
bharathts07 Oct 31, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions notebooks/ml-function-classify/notebook.ipynb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@
"metadata": {},
"outputs": [],
"source": [
"import json\n",
Copy link
Contributor Author

@bharathts07 bharathts07 Oct 31, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

import needed for indenting json output in notebook

"import seaborn as sns\n",
"import pandas as pd\n",
"import numpy as np\n",
Expand Down Expand Up @@ -366,7 +367,8 @@
"target_column: survival_status\n",
"description: \"Titanic passenger survival prediction based on demographics and ticket info\"\n",
"runtime: cpu-small\n",
"selected_features: {\"mode\":\"*\",\"features\":null}"
"selected_features: {\"mode\":\"*\",\"features\":null}\n",
"force: True"
Comment on lines +370 to +371
Copy link
Contributor Author

@bharathts07 bharathts07 Oct 31, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@syutika added this force command based on you suggestion

]
},
{
Expand All @@ -388,7 +390,7 @@
"outputs": [],
"source": [
"# Display the training result\n",
"training_result"
"print(json.dumps(training_result, indent=4))"
Copy link
Contributor Author

@bharathts07 bharathts07 Oct 31, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For better indentation in output

]
},
{
Expand Down