hammerlab / cohorts / 468

53%

push

travis-ci

first draft of strip_column_names utility function (#85)

* first draft of clean_column_names util

* fix example

* add tests

* fix tests

* first attempt to address comments

* more formatting changes

* including revised utils

* minor fixes

* add rename_cols arg to as_dataframe

* rename clean_column_names to strip_column_names

* add test cases for rename_cols in as_dataframe

* fix tests; some formatting

* tweak tests

* tweak formatting

* tests working

* fix d->df bug

* address comments

* add printed message, since warning not always displayed

* improve tests (to actually catch errors)

* make sure warning does print (logging.warning does not)

* fix test cases - have duplicate column names

* clarify doc

* update cols when rename_cols=True

* update test to use v2/3 set

* remove unused things

* fix tests

33 of 33 new or added lines in 2 files covered. (100.0%)

563 of 928 relevant lines covered (60.67%)

1.21 hits per line