Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

Conversation

@sanjeevghimire
Copy link
Contributor

  • removed output and creds from notebook

Copy link

@markstur markstur left a comment

Choose a reason for hiding this comment

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

Typo is back
Please fix the regression. Also see other comments, just questions that I might have ignored if it wasn't for the regression.

"metadata": {},
"source": [
"## 4c. Developing The Model with Random Forest\n",
"## 4c. Devloping The Model with Random Forest\n",

Choose a reason for hiding this comment

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

regression on the typo

"metadata": {},
"source": [
"## 4d. Developing The Model with Gradient Boosting\n",
"## 4d. Devloping The Model with Gradient Boosting\n",

Choose a reason for hiding this comment

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

here too

" \"instance_id\": '<INSTANCE ID>',\n",
" \"url\": \"<URL>\"\n",
"wml_credentials = { \n",
" \"apikey\": \"<api key>\",\n",

Choose a reason for hiding this comment

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

is this a regression or you prefer lower?

"# @hidden_cell\n",
"# idadb_fe06e3df5eef4ac0b1c246b40ed3db6e.close()\n",
"# To learn more about the ibmdby package, please read the documentation: http://pythonhosted.org/ibmdbpy/\n"
"# Connect to datasource\n"

Choose a reason for hiding this comment

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

not sure if you meant to delete the "PLEASE ENTER..." comment. Your readme probably has it covered.

@markstur markstur merged commit e697e80 into master Aug 23, 2019
@markstur markstur deleted the nb_output branch August 23, 2019 18:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants