Skip to content

[ENH] Upgrade OWSvm unittests#1499

Merged
tomazc merged 4 commits intobiolab:masterfrom
VesnaT:svm_tests
Sep 9, 2016
Merged

[ENH] Upgrade OWSvm unittests#1499
tomazc merged 4 commits intobiolab:masterfrom
VesnaT:svm_tests

Conversation

@VesnaT
Copy link
Copy Markdown
Contributor

@VesnaT VesnaT commented Aug 5, 2016

No description provided.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Aug 5, 2016

CLA assistant check
All committers have signed the CLA.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 5, 2016

Current coverage is 88.26% (diff: 100%)

Merging #1499 into master will not change coverage

@@             master      #1499   diff @@
==========================================
  Files            77         77          
  Lines          7624       7624          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           6729       6729          
  Misses          895        895          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update a30e0c2...f882602

@VesnaT VesnaT changed the title [ENH] Upgrade OWSvm unittests [WIP] Upgrade OWSvm unittests Aug 5, 2016
@VesnaT VesnaT changed the title [WIP] Upgrade OWSvm unittests [ENH] Upgrade OWSvm unittests Aug 8, 2016
kernel=kernel,
degree=self.degree,
gamma=self.gamma,
gamma=self.gamma if self.gamma else self._default_gamma,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the same as

self.gamma or self._default_gamma

@VesnaT VesnaT changed the title [ENH] Upgrade OWSvm unittests [WIP][ENH] Upgrade OWSvm unittests Aug 29, 2016
@VesnaT VesnaT changed the title [WIP][ENH] Upgrade OWSvm unittests [ENH] Upgrade OWSvm unittests Aug 29, 2016
@tomazc tomazc self-assigned this Sep 9, 2016
@tomazc tomazc merged commit 46961b4 into biolab:master Sep 9, 2016
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.

6 participants