Skip to content

Neural networks - #5

Open
pasliwa wants to merge 6 commits into
masterfrom
neural-networks
Open

Neural networks#5
pasliwa wants to merge 6 commits into
masterfrom
neural-networks

Conversation

@pasliwa

@pasliwa pasliwa commented Jun 13, 2017

Copy link
Copy Markdown
Collaborator

No description provided.

@magickris93

Copy link
Copy Markdown
Collaborator

@pasliwa

  • It would be a good idea to automate selection of column names. It may be problematic if input data set changes.
  • Make sure your code is properly indented. For example :
for el in possible_layer_combinations:
    clf = MLPClassifier(solver='lbfgs', alpha=1e-5,
                    hidden_layer_sizes=el, random_state=1)
  • You should get rid of empty cells.

@rlatawiec

Copy link
Copy Markdown
Collaborator
  • Make your code PEP8 compliant (mostly lines that are too long)
  • Add more validation metrics, accuracy alone doesn't paint the whole picture
  • I'd use a premade method for parameter tuning like RandomizedSearchCV from sklearn.model_selection

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.

3 participants