Skip to content

[ENH] t-SNE: Updates 2.#3475

Merged
janezd merged 3 commits intobiolab:masterfrom
VesnaT:tsne4
Dec 22, 2018
Merged

[ENH] t-SNE: Updates 2.#3475
janezd merged 3 commits intobiolab:masterfrom
VesnaT:tsne4

Conversation

@VesnaT
Copy link
Contributor

@VesnaT VesnaT commented Dec 14, 2018

Issue
Description of changes
Includes
  • Code changes
  • Tests
  • Documentation

@VesnaT VesnaT force-pushed the tsne4 branch 3 times, most recently from 7a5a19a to 0fc819f Compare December 14, 2018 13:01
@codecov
Copy link

codecov bot commented Dec 14, 2018

Codecov Report

Merging #3475 into master will increase coverage by 0.01%.
The diff coverage is 82.85%.

@@            Coverage Diff             @@
##           master    #3475      +/-   ##
==========================================
+ Coverage   83.49%   83.51%   +0.01%     
==========================================
  Files         367      367              
  Lines       65282    65300      +18     
==========================================
+ Hits        54509    54533      +24     
+ Misses      10773    10767       -6

@codecov
Copy link

codecov bot commented Dec 14, 2018

Codecov Report

Merging #3475 into master will increase coverage by 0.02%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##           master    #3475      +/-   ##
==========================================
+ Coverage   83.55%   83.58%   +0.02%     
==========================================
  Files         367      367              
  Lines       65454    65476      +22     
==========================================
+ Hits        54690    54725      +35     
+ Misses      10764    10751      -13

@VesnaT VesnaT changed the title [WIP] t-SNE: Updates 2. t-SNE: Updates 2. Dec 20, 2018
@VesnaT VesnaT changed the title t-SNE: Updates 2. [ENH] t-SNE: Updates 2. Dec 20, 2018

#: Runtime state
Running, Finished, Waiting = 1, 2, 3
Running, Finished, Waiting, Pending = 1, 2, 3, 4
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe "Pending" isn't the greatest name for what this indicates. A better name would be "Paused". Which kind of begs the question of what "Waiting" means...

@pavlin-policar
Copy link
Collaborator

@janezd This PR should be merged, but I don't have permissions to merge unless all the checks pass. Could you please use your superpower and press merge?

@janezd janezd merged commit a1f701a into biolab:master Dec 22, 2018
@janezd
Copy link
Contributor

janezd commented Dec 22, 2018

Sure. Thanks.

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.

3 participants