Skip to content

Polished code, added a dataset, improved the readability#2

Open
T4ras123 wants to merge 1 commit into
Suji04:masterfrom
T4ras123:master
Open

Polished code, added a dataset, improved the readability#2
T4ras123 wants to merge 1 commit into
Suji04:masterfrom
T4ras123:master

Conversation

@T4ras123

Copy link
Copy Markdown

Changes made:

  1. Added a iris.csv dataset to the directory so code in decision_tree_classification.ipynb can run out of the box.
  2. Removed num_samples parameter from the get_best_split() function as it is not used.
  3. Added f-strings and is not None in the if statements as a better practice.
  4. Impoved redability by fixing spacing following pep8.

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.

1 participant