Skip to content

Commit

Permalink
Make scrutinizer happy with newer pep8 version
Browse files Browse the repository at this point in the history
  • Loading branch information
@nouiz @dmitriy-serdyuk
nouiz authored and dmitriy-serdyuk committed Sep 6, 2018
1 parent e2d28d8 commit c85566a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion doctests/__init__.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ def load_tests(loader, tests, ignore):
module=importlib.import_module(module), setUp=setup,
optionflags=doctest.IGNORE_EXCEPTION_DETAIL,
checker=Py23DocChecker()))
except:
except Exception:
pass

# This part loads the doctests from the documentation
Expand Down
2 changes: 1 addition & 1 deletion fuel/converters/base.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ def fill_hdf5_file(h5file, data):
for name in split_names:
lengths = [len(split_tuple[2]) for split_tuple in data
if split_tuple[0] == name]
if not all(l == lengths[0] for l in lengths):
if not all(le == lengths[0] for le in lengths):
raise ValueError("split '{}' has sources that ".format(name) +
"vary in length")

Expand Down

1 comment on commit c85566a

@vrodr1guez
Copy link

Choose a reason for hiding this comment

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

def fill_hdf5_file(h5file, data):
split_names = list(set([split_tuple[0] for split_tuple in data]))
for split_name in split_names:
lengths = [len(split_tuple[2]) for split_tuple in data if split_tuple[0] == split_name]
if not all(le == lengths[0] for le in lengths):
raise ValueError("split '{}' has sources that ".format(split_name) + "vary in length")

Please sign in to comment.