Skip to content

Commit

Permalink
Merge pull request #204 from broadinstitute/sn_update_logging
Browse files Browse the repository at this point in the history
Adding logging
  • Loading branch information
snovod authored Jan 29, 2025
2 parents ab99565 + a60735c commit 92f55ea
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions python/generate_tdr_schema_json.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,14 @@
from utils.tdr_utils.tdr_schema_utils import InferTDRSchema
from utils.terra_utils.terra_util import TerraWorkspace
from utils.token_util import Token
import logging

CLOUD_TYPE = GCP

logging.basicConfig(
format="%(levelname)s: %(asctime)s : %(message)s", level=logging.INFO
)


def get_args() -> argparse.Namespace:
parser = argparse.ArgumentParser(description="Get TDR schema JSON file")
Expand Down Expand Up @@ -53,15 +58,15 @@ def get_args() -> argparse.Namespace:
args = parser.parse_args()

# Custom validation logic
workspace_args = [args.billing_project, args.workspace_name, args.terra_table_names]
workspace_args = [args.billing_project, args.workspace_name, args.terra_table_names] # type: ignore[assignment]
if args.input_tsv:
if any(workspace_args):
if any(workspace_args): # type: ignore[arg-type]
parser.error(
"Cannot provide BOTH input_tsv AND the combination of billing_project, workspace_name, "
"and terra_table_names."
)
else:
if not all(workspace_args):
if not all(workspace_args): # type: ignore[arg-type]
parser.error(
"If input_tsv is not provided, you must provide ALL of workspace_name, billing_project, "
"and terra_table_names."
Expand Down

0 comments on commit 92f55ea

Please # to comment.