Skip to content

Commit

Permalink
fixup: appease pylint
Browse files Browse the repository at this point in the history
  • Loading branch information
rileyhgrant committed Feb 29, 2024
1 parent aef33ba commit 36c5185
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
1 change: 1 addition & 0 deletions data_pipeline/data_pipeline/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,5 +18,6 @@
for section, option in REQUIRED_CONFIGURATION:
value = pipeline_config.get(section, option)
assert value
# pylint: disable=broad-exception-raised
except (configparser.NoOptionError, AssertionError) as exc:
raise Exception(f"Missing required configuration '{section}.{option}'") from exc
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,12 @@ def prepare_variant_results():
an_case=results.ac_case[0],
an_ctrl=results.ac_control[0],
)


# pylint: disable=fixme
# TODO: I also should do some renaming and filtering here:
# the three sub-things I have coming in are lowercase, and end in -control


# pylint: disable=broad-exception-raised
# TODO: also, in gene results I should figure out what is going on with all the
# bajillion fields I'm returning (0_001_03, etc)
# need to check the input schema of something like Epi25 vs IBD
Expand Down
1 change: 1 addition & 0 deletions data_pipeline/write_results_files.py
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ def split_data(row):

def write_data_files(table_path, output_directory, genes=None):
if output_directory.startswith("gs://"):
# pylint: disable=broad-exception-raised
raise Exception("Cannot write output to Google Storage")

ds = hl.read_table(table_path)
Expand Down

0 comments on commit 36c5185

Please sign in to comment.