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.

Sign up for GitHub

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

Jump to bottom

merging backwards to daily-dev #74

Closed
wants to merge 7 commits into from
Closed

merging backwards to daily-dev #74

wants to merge 7 commits into from

Conversation

mikkokotila
Copy link
Contributor

No description provided.

ackRow and others added 7 commits July 30, 2018 02:13
@ackRow
Functional Model Support v2
026a5a5 
Substitute `model.predict_classes`  (only available for Sequential Model) using `numpy.argmax` with `axis=-1` to match predict_classes shape.
@guilhermesilveira
fixing the notebook URI
7540674
@mikkokotila
Merge pull request #58 from guilhermesilveira/patch-1
9188981 
fixing the notebook URI
@ackRow
Merge pull request #1 from autonomio/dev
6a6aab8 
Syncing with original repo
Experimental Support for Functional Model
c7519c7 
Using a predict_classes function similar to Sequential Model for Functional Model

To use this experimental function add experimental_functional_support=True to the Scan() parameters

Adding a Jupyter Notebook example
@mikkokotila
Merge pull request #70 from ackRow/dev
d9b2c0b 
Adding Experimental Support for Functional Model
@mikkokotila
Merge pull request #71 from autonomio/daily-dev
b07cd68 
fixes #62 and adds several missing docstrings
@pep8speaks
Copy link

Hello @mikkokotila! Thanks for submitting the PR.

Line 6:69: W291 trailing whitespace
Line 37:80: E501 line too long (105 > 79 characters)
Line 39:1: W293 blank line contains whitespace
Line 45:1: W293 blank line contains whitespace
Line 45:1: W391 blank line at end of file

@coveralls
Copy link

coveralls commented Aug 22, 2018 •
edited

Pull Request Test Coverage Report for Build 169

  • 5 of 13 (38.46%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.5%) to 88.366%
Changes Missing Coverage Covered Lines Changed/Added Lines %
talos/metrics/score_model.py 4 12 33.33%
Totals Coverage Status
Change from base Build 166: -0.5%
Covered Lines: 714
Relevant Lines: 808
💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@mikkokotila @pep8speaks @coveralls @ackRow @guilhermesilveira