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

v.0.6 to Dev #339

Merged
merged 25 commits into from
Jul 5, 2019
Merged

v.0.6 to Dev #339

merged 25 commits into from
Jul 5, 2019

Conversation

mikkokotila
Copy link
Contributor

See commit details for more information.

This version will introduce a self-contained ParamSpace API allowing Scan() a single line interface to qualified permutations. The change allows important streamlining of the main procedural codes of Talos. These simplifications are already reflected in this commit.

Other changes:

- a major rehaul of reducers; reduction strategies can now be easily added in a single file/function
- it's now very
- moved all logging/results related codes to /logging
- Deals with #153
- Major cleaning up of Scan() arguments
- Adds a reducer that takes in a metric, threshold, and loss e.g ['val_acc', 0.9, False] where the experiment will be ended once a given metric threshold is met by a model
- /examples is now /templates
- many redundant functions / files were deleted
- ~100 lines of code were removed from the mainline codes and those codes were notably streamlined
- reduction no longer has prepare or finish
- learning entropy (metrics/entropy.py) is completely rewritten

A big bunch of other things, so do check it out for yourself.

NOTE: this version is still under testing.
- KerasModel now accepts 'task' argument which can be either prediction task type or a list ot metrics. If prediction task name (e.g. binary) is passed, metric is automatically picked.
- Params ranged values after editing the dictionary are now lists and not ranges
- Removed permutation_filter as this is now handled in ParamSpace remove_lambda()
- changed clear_tf_session to clear_session to support any backend
- changed the previous tf specific session clearing to universal one
- added a couple of metrics and cleaned up the metrics names
- fixes KerasModel in a way where all prediction task types are supported
- added output_layer for infering the right activation and last neuron count
- renamed layers.py to hidden_layers.py
v.0.5 to Master (ACTUAL)
- removed the option for metric to be logged at peak epoch (now always last epoch is recorded for consistency with Keras)
- this means that also the record of which epoch was the peak is removed (for now, later this will be added back)
- made some changes to logging in an effort to remove all kinds of mystique that remains in it
- cleaned up codes
This is related with the cases where < 3.6 python can cause various logging related problems, such as the #273 #267 and #255.
- handles the error resulting form matplotlib on some cases with mac os x and virtualenv
- removes stringification of the results and resolve #271
- the above also makes type handling in scan_finish redundant and is removed
From v.0.6 onwards, Talos requires python >= 3.5. For older versions of python use, Talos v.0.5 which is the previous LTS.
@pep8speaks
Copy link

Hello @mikkokotila! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 75:1: W293 blank line contains whitespace

Line 71:9: E722 do not use bare 'except'

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

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

Line 44:9: E722 do not use bare 'except'

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

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

@mikkokotila mikkokotila changed the title v.0.6 to Daily Dev v.0.6 to Dev Jul 5, 2019
@mikkokotila mikkokotila merged commit 8f38a4a into dev Jul 5, 2019
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.

2 participants