awslabs / amazon-omics-tools

Apache License 2.0
18 stars 6 forks source link

(omics.cli.run_analyzer) storageType is null for old runs. #45

Closed hmkim closed 1 month ago

hmkim commented 1 month ago

In my case, the run has not the storageType of res variable.

$ python -m omics.cli.run_analyzer 7379101 -o run_7379101.csv Traceback (most recent call last): File "", line 198, in _run_module_as_main File "", line 88, in _run_code File "/Users/hyunmink/miniforge3/envs/boto3/lib/python3.12/site-packages/omics/cli/run_analyzer/main.py", line 489, in add_metrics(res, resources, pricing) File "/Users/hyunmink/miniforge3/envs/boto3/lib/python3.12/site-packages/omics/cli/run_analyzer/main.py", line 339, in add_metrics metrics["sizeReserved"] = f"{capacity} GiB" ^^^^^^^^ UnboundLocalError: cannot access local variable 'capacity' where it is not associated with a value

Screenshot 2024-08-05 at 12 34 05 PM
hmkim commented 1 month ago

This appears to be an issue with runs that were probably run before HeathOmics announced support for the dynamic run storage type.

https://aws.amazon.com/about-aws/whats-new/2024/04/aws-healthomics-dynamic-run-storage-private-workflows/