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.2 to Dev #362

Merged
merged 14 commits into from
Aug 3, 2019
Merged

v.0.6.2 to Dev #362

merged 14 commits into from
Aug 3, 2019

Conversation

mikkokotila
Copy link
Contributor

See individual commits for more details.

This commit initiates v0.6.1

- Removed the default 'val_acc' so instead have to explicitly state metric when deploying (fixes #283)
- Fixed a bug related with the case where x is not 2d
- Fixed test_script.py to correspond with the changes
The most import aspect of this version update is the new docs. The docs are comprehensive, and consistent with the >0.6 Talos API.

Some small updates and fixes outside of the docs are included in this update.

- Reporting() now also accepts Analyze() as its command
- Reporting() no longer has 'val_acc' as default value in any of the properties
- early_stopper() now has 'lazy' mode with slight tweaks to other modes
- early_stopper() no longer expects epochs value for custom settings
- Scan() docstrings are up-to-date and cleaned up
- some other docstring cleanups are present
- Docstrings will take the same format in the future as markup table to give consistency with documentations

NOTE: the docs are a work in progress and will be completed throughout this week. Pushing now to avoid the case where for some reason several days of work will be lost.
- there is now a new sub-module `autom8` inside which several AutoML features live
- `AutoParams` automatically generates a parameter dictionary and streamlines its manipulation before experiment
- `AutoModel` automatically creates a input model for `Scan()` which is fully wired for use with `AutoParams` or other experiment with comprehensive search
- `AutoScan` leverage `AutoParams` and `AutoModel` to reduce the whole experiment into a single line of code
- `AutoPredict` takes the results of `Scan` (or `AutoScan`), picks best model candidates, evaluates the candidates, picks the winner, and makes predictions with it on input data
- the new docs are now completed
- added `local_strategy` to reduction strategies, which allows making changes to the parameter space from local system while the experiment is running
- added `pearson` and `kendall` reduction strategies
- streamlined the way custom strategies can be added
- completely rebuilt `correlation` strategy, including the underlying statistical approach
- added a helper function `cols_to_multilabel` for custom reducers
- added a new generator `SequenceGenerator`
- removed redundant files from the repo
- tests are updated in regards to the changes but not yet new features
@pep8speaks
Copy link

pep8speaks commented Aug 2, 2019

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

Line 6:1: E402 module level import not at top of file
Line 7:1: E402 module level import not at top of file
Line 16:1: E402 module level import not at top of file

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

Line 1:1: E302 expected 2 blank lines, found 0
Line 100:80: E501 line too long (84 > 79 characters)
Line 103:80: E501 line too long (82 > 79 characters)

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

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

Line 31:19: E712 comparison to True should be 'if cond is True:' or 'if cond:'

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

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

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

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

Line 66:80: E501 line too long (97 > 79 characters)
Line 151:5: E303 too many blank lines (2)

Comment last updated at 2019-08-03 13:07:05 UTC

- added `talos.utils.ExperimentLogCallback` which allows storing epoch-by-epoch training data on the local machine during the experiment (implements the request in #153)
- added "edit on github" link to docs
- added free text search to docs
- added some styling to docs
- added analytics to docs
- added "copy to clipboard" to code snippets in docs
- added some tests
...is more than just a catchy phrase.

In any case, the test suite is now clean and extensible as opposed to the coverage obsessed mess that it was before.

Quite a few bugs were squashed betwee now and Friday 4pm. Most importantly, there were quite a few sloppiness in the reducer process, that would have effected the occassional case.
@mikkokotila mikkokotila merged commit 4bb792a into dev Aug 3, 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