Skip to content

Commit

Permalink
Fix issue with undeterministic test case
Browse files Browse the repository at this point in the history
  • Loading branch information
has2k1 committed Jul 31, 2018
1 parent 0ab88f2 commit 83183df
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion plotnine/scales/scale.py
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ def train_df(self, df):
"""
Train scale from a dataframe
"""
aesthetics = set(self.aesthetics) & set(df.columns)
aesthetics = sorted(set(self.aesthetics) & set(df.columns))
for ae in aesthetics:
self.train(df[ae])

Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 0 additions & 2 deletions plotnine/tests/test_scale_internals.py
Original file line number Diff line number Diff line change
Expand Up @@ -402,8 +402,6 @@ def scale_scale_discrete_mapping_nulls():
assert res[2] == expected[2]


@pytest.mark.skipif(
six.PY2, reason="I do not know why it fails on travis")
def test_multiple_aesthetics():

df = pd.DataFrame({
Expand Down

0 comments on commit 83183df

Please sign in to comment.