search

ImagingDataCommons / ETL

(CORE REPO)
Apache License 2.0
0 stars 1 forks source link

Consider adding instance size to the BQ tables #4

Closed fedorov closed 2 years ago

fedorov commented 4 years ago

It will probably be quite valuable to the users to know how much space the cohort will take when downloaded. In particular, this will be important for the user in order to allocate sufficient disk space when downloading the data to the VM.

I think we should add a column with the size for each instance we are indexing. We discussed this with @wlongabaugh and he also thought it makes sense. What do you think @bcli4d?

wlongabaugh commented 4 years ago

And provide the sum as part of the interactive cohort download.

bcli4d commented 4 years ago

Instance size is in the auxiliary metadata table. Are you asking for it to be added to dicom_all? We can do. In the next update after the Tuesday release?

fedorov commented 4 years ago

I missed that it is in the aux table - this is great! Yes, I think it makes sense to add it to dicom_all on the next update, and coordinate with @s-paquette to provide this information to the user in the portal UI.

fedorov commented 2 years ago

instance_size has been added to dicom_all.