Skip to content

Commit

Permalink
ex-236 (cgates): adjust glossary to work in py3
Browse files Browse the repository at this point in the history
  • Loading branch information
cgates committed May 5, 2015
1 parent 2157808 commit 580ae50
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion jacquard/expand.py
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ def validate_args(args):
def _get_actual_columns(vcf_reader, col_spec):
columns = _create_potential_column_list(vcf_reader)
glossary_fields = []
for glossary_field in sorted(columns.values()):
for glossary_field in sorted([x for x in columns.values() if x]):
if glossary_field and glossary_field not in glossary_fields:
glossary_fields.append(glossary_field)

Expand Down
2 changes: 1 addition & 1 deletion test/expand_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ def test_create_potential_column_list_preservesSampleOrdering(self):
sample_names=sample_names)

actual_cols = expand._create_potential_column_list(mock_vcf_reader)
actual_format_sample_names = actual_cols.keys()[8:]
actual_format_sample_names = list(actual_cols.keys())[8:]

expected_format_sample_names = ["A|sample1", "A|sample2", "A|sample10",
"B|sample1", "B|sample2", "B|sample10"]
Expand Down

0 comments on commit 580ae50

Please sign in to comment.