Daily dev

[mikkokotila]

• fixed results.py causes some metrics to take the lowest value from history object #62 (does not change the mainline in anyway)
• added docstrings

[pep8speaks]

Hello @mikkokotila! Thanks for updating the PR.

• In the file talos/metrics/performance.py, following are the PEP8 issues :

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

• In the file talos/model/shapes.py, following are the PEP8 issues :

Line 26:5: E741 ambiguous variable name 'l'
Line 57:13: E741 ambiguous variable name 'l'
Line 71:13: E741 ambiguous variable name 'l'
Line 111:80: E501 line too long (94 > 79 characters)
Line 117:80: E501 line too long (94 > 79 characters)
Line 141:13: E741 ambiguous variable name 'l'
Line 160:13: E741 ambiguous variable name 'l'
Line 176:9: E741 ambiguous variable name 'l'

Comment last updated on August 20, 2018 at 14:58 Hours UTC

[matthewcarbone]

@mikkokotila regarding

the WIKI needs to updated related with custom metrics (the word 'acc') needs to be in a metric always

This would pose an issue if the user wanted to use the F1 score though, right?

Also side question: what is the daily-dev branch? 😄

[coveralls]

Pull Request Test Coverage Report for Build 161

• 9 of 9 (100.0%) changed or added relevant lines in 2 files are covered.
• 3 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.2%) to 88.962%

---

Files with Coverage Reduction	New Missed Lines	%
talos/metrics/performance.py	3	83.78%

Totals
Change from base Build 156:	-0.2%
Covered Lines:	677
Relevant Lines:	761

---

💛 - Coveralls