ImagingDataCommons / ETL

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

Missing mergeschemas.py #1

Closed fedorov closed 4 years ago

fedorov commented 4 years ago

mergeschemas.py then merges all these schemas into one schema that defines the dicom_all table.

@G-White-ISB I just noticed this file is missing from the repo. Can you add it for the sake of completeness?

s-paquette commented 4 years ago

Before it's added to the repo, can I get an understanding of what it is? (We don't typically add one-off use scripts to the repo long-term if they're for building a temporary set of data; we'll archive them into Google Drive or add them to an ETL repo.)

G-White-ISB commented 4 years ago

Hi, So Suzanne we had already created a separate repo just for putting in ETL: https://github.com/ImagingDataCommons/ETL. mergeschemas.py has been added. Geirge

On Mon, Feb 3, 2020 at 12:08 PM S. Paquette notifications@github.com wrote:

Before it's added to the repo, can I get an understanding of what it is? (We don't typically add one-off use scripts to the repo long-term if they're for building a temporary set of data; we'll archive them into Google Drive or add them to an ETL repo.)

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/ImagingDataCommons/ETL/issues/1?email_source=notifications&email_token=AN54OR2APNYFENVWEWLEAF3RBB2T3A5CNFSM4KPKHYX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKVG5SY#issuecomment-581594827, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN54OR2CMSSGOJH2XJGXKR3RBB2T3ANCNFSM4KPKHYXQ .

s-paquette commented 4 years ago

Perfect!

fedorov commented 4 years ago

Thank you!