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

weekly daily-dev to dev merge #90

Merged
merged 29 commits into from
Oct 1, 2018
Merged

weekly daily-dev to dev merge #90

merged 29 commits into from
Oct 1, 2018

Conversation

mikkokotila
Copy link
Contributor

See individual commits for more.

regular dev to master merge
- added stratified sampling
- added latin hypercube sampling with sudoku constraint
- added latin hypercub e
- commented several functions
- moved shuffle as an option in sample_reducer
- added 'random_method' parameter to Scan()
- cleaned up and modified README.md
- added latin hypercube sampling with sudoku constraint submodule
- added the new features to tests
# the new Randomizer() supports:

- Quantum randomness (vacuum based)
- Ambient Sound based randomness
- Sobol sequences
- Halton sequences
- Latin hypercube
- Improved Latin hypercube
- Latin hypercube with a Sudoku-style constraint
- Uniform Mersenne
- Cryptographically sound uniform
- Reduced testing script rounds to 5 from 10
- FIX: a bug in sample_reduce that prevented stratification from working
- TODO: need to think how stratification works with various randomizers and if it makes sense to have a separate option or random_method as it is now

Also updated the README.md
@pep8speaks
Copy link

pep8speaks commented Sep 8, 2018

Hello @mikkokotila! Thanks for updating the PR.

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

Line 5:1: W293 blank line contains whitespace

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

Line 61:80: E501 line too long (126 > 79 characters)
Line 67:80: E501 line too long (94 > 79 characters)

Line 7:1: E302 expected 2 blank lines, found 1
Line 23:80: E501 line too long (102 > 79 characters)

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

Comment last updated on October 01, 2018 at 21:17 Hours UTC

@coveralls
Copy link

coveralls commented Sep 8, 2018

Pull Request Test Coverage Report for Build 234

  • 76 of 144 (52.78%) changed or added relevant lines in 12 files are covered.
  • 4 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-5.04%) to 83.263%

Changes Missing Coverage Covered Lines Changed/Added Lines %
talos/metrics/score_model.py 7 8 87.5%
talos/model/layers.py 2 3 66.67%
talos/scan/scan_prepare.py 5 7 71.43%
talos/utils/reporting.py 1 3 33.33%
talos/scan/scan_run.py 10 15 66.67%
talos/utils/tf_config.py 0 6 0.0%
talos/reducers/sample_reducer.py 24 33 72.73%
talos/utils/validation_split.py 2 16 12.5%
talos/utils/predict.py 12 40 30.0%
Files with Coverage Reduction New Missed Lines %
talos/utils/reporting.py 1 89.13%
talos/metrics/performance.py 3 83.78%
Totals Coverage Status
Change from base Build 185: -5.04%
Covered Lines: 786
Relevant Lines: 944

💛 - Coveralls

mikkokotila and others added 14 commits September 20, 2018 23:43
fix a missing SSL cert problem for quantum random number API call
regular master >> daily-dev
- improved / added docstrings
- pepified
- added Randomizer as a stand-alone package
- turned "clear_tf_session" True by default in Scan()
- added tf_config function to /utils as a new home for tf resource management functionality
This is now moved into its own package in a separate repo and installs with 'pip install chances'.
TrainingLog adds a callback function with live view of the training process against any metric.
- pepification
- moved 'model' to the first parameters in Scan()
- made 'dataset_name' and 'experiment_no' optional
- 'dataset_name' becomes timestamp with second accuracy if None
- added 'print_params' option which prints the params every round
- created custom exceptions / error handling for the most common (and annoying) cases
- added an experimental best model saving
- created exceptions.py in /utils
- fixed an issue in reporting.py with table() where both metric and sort_by needed to be separately stated unless metric was val_acc
- creates two new objects 'saved_weights' and 'saved_models'
- stores the model (as json) and the model weights for each round
- added new class Predict() in /utils/predict.py
- added kfold splitting to /utils/validation_split.py
- added f1-score based model validation
@mikkokotila mikkokotila merged commit 2a9c57a into dev Oct 1, 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

4 participants