Skip to content
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.

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

Version Update to PyPi #8

Merged
merged 57 commits into from
May 14, 2018
Merged

Version Update to PyPi #8

merged 57 commits into from
May 14, 2018

Conversation

mikkokotila
Copy link
Contributor

No description provided.

C0NTRIBUTE and others added 30 commits May 7, 2018 14:39
The codebase is completely cleaned up and separated into sub-modules in the way that makes most sense from the future development standpoint.
- added several datasets (cervical cancer, titanic, iris)
- added a multi-classification model
- added to spear_reductiong handling and user message for the case where previous interval only produce nan values (usually means there is a nan in x)
- removed some debuggin codes from scan
- added several new datasets
- added BinaryPerformance for unified scoring
- added the 'missing' Keras metrics fmeasure, etc.
- added the ability to change the monitor for early stop
- reporting focus now on summarizing the best and the worst
- round results are now produced in a far more cleaner way that allow using any metric, and also cases where acc and loss are not used
- added saving of model (not yet saving the best model though)
- cleaned up the template from redundant items
- reduced dependy to template.py in a push to make it entire redudant
- renamed same files
- now performance metric is unified for binary, multi-label, and multi-class
- spear allows any reduction_metric
- added pred_class() which automatically detects the type of prediction challenge
- cleaned up results
- Improved the way that reporting handles change in labels
- Made templates.py completely redundant as part of the wider effort to make the entire pipeline completely dynamic in the sense of not relying at all in string labels
- Fixed a bug in spear_reducer that locked the changes to learning rate only
- Changing the name to Talos
- added shapes
- exposed all Dense layer parameters from hidden_layers() layer generator
- changed data folder name to examples
- cleaned up the namespace
- fixed issues in the Reporting with the way the function names are handled
- added a breast_cancer model to examples
- added iris and breast_cancer params to examples
- renamed the /examples to /notebooks
- there was an error being raised when no shape was set
- the detector function was missing an import
- results was in every case reporting the last epoch as peak result
- early_stopper did not technically allow custom inputs for min_delta and patience
This will be part of the "generality" metric, so that talos optimizers will have to optimize against performance and generality at the same time, to avoid finding just high performance scores that will not generalize well.
- astetik plots are now available through ta.plots...
…ptimization on Keras with Breast Cancer Data.ipynb
@pep8speaks
Copy link

Hello @mikkokotila! Thanks for submitting the PR.

  • In the file setup.py, following are the PEP8 issues :

Line 27:1: E302 expected 2 blank lines, found 1
Line 53:5: E303 too many blank lines (2)
Line 55:1: E305 expected 2 blank lines after class or function definition, found 1
Line 83:80: E501 line too long (83 > 79 characters)
Line 84:80: E501 line too long (82 > 79 characters)
Line 89:1: E124 closing bracket does not match visual indentation

Line 6:80: E501 line too long (88 > 79 characters)
Line 30:80: E501 line too long (83 > 79 characters)
Line 57:80: E501 line too long (84 > 79 characters)

Line 13:80: E501 line too long (105 > 79 characters)
Line 18:80: E501 line too long (101 > 79 characters)
Line 36:80: E501 line too long (114 > 79 characters)
Line 43:80: E501 line too long (90 > 79 characters)
Line 47:80: E501 line too long (101 > 79 characters)

Line 10:24: E231 missing whitespace after ':'
Line 11:25: E231 missing whitespace after ':'
Line 15:28: E231 missing whitespace after ':'
Line 17:17: E231 missing whitespace after ':'
Line 17:26: E231 missing whitespace after ','
Line 20:22: E231 missing whitespace after ':'
Line 30:24: E231 missing whitespace after ':'
Line 31:25: E231 missing whitespace after ':'
Line 35:28: E231 missing whitespace after ':'
Line 37:17: E231 missing whitespace after ':'
Line 37:26: E231 missing whitespace after ','
Line 40:22: E231 missing whitespace after ':'

Line 7:80: E501 line too long (80 > 79 characters)

Line 79:80: E501 line too long (86 > 79 characters)

Line 32:21: E721 do not compare types, use 'isinstance()'

Line 54:5: E303 too many blank lines (2)

Line 1:80: E501 line too long (81 > 79 characters)

Line 13:5: E741 ambiguous variable name 'l'
Line 44:13: E741 ambiguous variable name 'l'
Line 58:13: E741 ambiguous variable name 'l'
Line 98:80: E501 line too long (94 > 79 characters)
Line 104:80: E501 line too long (94 > 79 characters)
Line 128:13: E741 ambiguous variable name 'l'
Line 147:13: E741 ambiguous variable name 'l'
Line 163:9: E741 ambiguous variable name 'l'

Line 72:41: E721 do not compare types, use 'isinstance()'

Line 13:30: E711 comparison to None should be 'if cond is None:'
Line 17:80: E501 line too long (84 > 79 characters)
Line 20:80: E501 line too long (88 > 79 characters)
Line 41:80: E501 line too long (92 > 79 characters)
Line 41:89: E231 missing whitespace after ','
Line 42:80: E501 line too long (82 > 79 characters)

Line 26:74: E231 missing whitespace after ','
Line 26:80: E501 line too long (101 > 79 characters)
Line 26:97: E231 missing whitespace after ','
Line 40:80: E501 line too long (82 > 79 characters)
Line 50:80: E501 line too long (107 > 79 characters)
Line 50:103: E231 missing whitespace after ','
Line 53:80: E501 line too long (106 > 79 characters)
Line 53:102: E231 missing whitespace after ','

Line 5:80: E501 line too long (88 > 79 characters)
Line 11:80: E501 line too long (85 > 79 characters)
Line 36:18: E712 comparison to True should be 'if cond is True:' or 'if cond:'
Line 91:80: E501 line too long (85 > 79 characters)
Line 107:28: E712 comparison to True should be 'if cond is True:' or 'if cond:'
Line 114:5: E303 too many blank lines (2)

Line 17:80: E501 line too long (86 > 79 characters)

Line 21:41: E721 do not compare types, use 'isinstance()'

Line 12:21: E712 comparison to True should be 'if cond is True:' or 'if cond:'

Line 9:80: E501 line too long (81 > 79 characters)
Line 34:80: E501 line too long (101 > 79 characters)
Line 62:80: E501 line too long (101 > 79 characters)
Line 71:80: E501 line too long (107 > 79 characters)
Line 77:1: E303 too many blank lines (3)
Line 81:1: E305 expected 2 blank lines after class or function definition, found 3
Line 87:80: E501 line too long (80 > 79 characters)
Line 97:16: E127 continuation line over-indented for visual indent
Line 98:16: E127 continuation line over-indented for visual indent
Line 99:16: E127 continuation line over-indented for visual indent
Line 100:16: E127 continuation line over-indented for visual indent
Line 101:16: E127 continuation line over-indented for visual indent
Line 102:16: E127 continuation line over-indented for visual indent
Line 103:16: E127 continuation line over-indented for visual indent
Line 111:20: E231 missing whitespace after ':'
Line 112:21: E231 missing whitespace after ':'
Line 112:24: E231 missing whitespace after ','
Line 112:26: E231 missing whitespace after ','
Line 112:28: E231 missing whitespace after ','
Line 116:24: E231 missing whitespace after ':'
Line 120:18: E231 missing whitespace after ':'
Line 124:12: E127 continuation line over-indented for visual indent
Line 125:12: E127 continuation line over-indented for visual indent
Line 126:12: E127 continuation line over-indented for visual indent
Line 127:12: E127 continuation line over-indented for visual indent
Line 128:12: E127 continuation line over-indented for visual indent

@mikkokotila mikkokotila merged commit 10f460e into production May 14, 2018
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.

None yet

3 participants