Skip to content

[FIX] Table.__repr__: Fix for sparse data with < 5 instances#1951

Merged
lanzagar merged 1 commit intobiolab:masterfrom
nikicc:repr_on_small_sparse
Jan 25, 2017
Merged

[FIX] Table.__repr__: Fix for sparse data with < 5 instances#1951
lanzagar merged 1 commit intobiolab:masterfrom
nikicc:repr_on_small_sparse

Conversation

@nikicc
Copy link
Contributor

@nikicc nikicc commented Jan 25, 2017

Issue

Fixes #1950.

Description of changes

Fix Table.__repr__ for sparse.

Includes
  • Code changes
  • Tests
  • Documentation

@nikicc nikicc force-pushed the repr_on_small_sparse branch from 317aa53 to 5ee3193 Compare January 25, 2017 09:58
@astaric astaric added this to the 3.3.11 milestone Jan 25, 2017
@astaric
Copy link
Member

astaric commented Jan 25, 2017

A test would be nice :)

Scipy does not support indexing table[:5] if table has less than 5 rows.
@nikicc nikicc force-pushed the repr_on_small_sparse branch from 5ee3193 to c6f94f2 Compare January 25, 2017 10:43
@nikicc
Copy link
Contributor Author

nikicc commented Jan 25, 2017

Simple test was added. Also, checking for sparse was replaced with self.is_sparse().

@codecov-io
Copy link

codecov-io commented Jan 25, 2017

Current coverage is 89.44% (diff: 100%)

Merging #1951 into master will increase coverage by 0.11%

@@             master      #1951   diff @@
==========================================
  Files            86         86          
  Lines          9119       9122     +3   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           8146       8159    +13   
+ Misses          973        963    -10   
  Partials          0          0          

Sunburst

Powered by Codecov. Last update af3cb70...c6f94f2

@lanzagar lanzagar merged commit 3878b7b into biolab:master Jan 25, 2017
@nikicc nikicc deleted the repr_on_small_sparse branch January 25, 2017 12:39
astaric pushed a commit that referenced this pull request Feb 3, 2017
[FIX] Table.__repr__: Fix for sparse data with < 5 instances
(cherry picked from commit 3878b7b)
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.

4 participants