Skip to content

Commit

Permalink
Merge branch '52-setup-service-decouple-from-dane' into 81-prov
Browse files Browse the repository at this point in the history
  • Loading branch information
greenw0lf authored Sep 26, 2024
2 parents 5def84b + ecb2eb5 commit d541866
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion download.py
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,11 @@ def s3_download(s3_uri: str) -> Optional[DownloadResult]:
if not success:
logger.error("Failed to download input data from S3")
return None
download_time = (time.time() - start_time) * 1000 # time in ms

download_time = int((time.time() - start_time) * 1000) # time in ms
else:
download_time = -1 # Report back?

provenance = {
"activity_name": "Input download",
"activity_description": "Downloads the input file from INPUT_URI",
Expand All @@ -113,6 +117,7 @@ def s3_download(s3_uri: str) -> Optional[DownloadResult]:
"output_data": input_file,
"steps": [],
}

return DownloadResult(
input_file, mime_type, provenance, download_time # TODO add content_length
)

0 comments on commit d541866

Please sign in to comment.