Open fedorov opened 1 year ago
Can someone confirm the correctness of this file? There are a few sheets that don't have any data. https://github.com/ImagingDataCommons/IDC-ProjectManagement/files/12196403/CCDI_Submission_Template_v1.0.1_DM_v2.xlsx
@G-White-ISB There were different versions of the spreadsheet. Attached is the one I believe to be the latest version after having reviewed the #1394 issue. CCDI_Submission_Template_v1.0.1_DM_v2 2023-02-17.xlsx
For your information, David C. and I met today with the Childhood Cancer Data Initiative (CCDI) folks. They may send us another version of the spreadsheet but the timing is undetermined. Therefore, I propose that you go forward with the attached spreadsheet and we update the clinical data if and when we get a richer set of data from the CCDI folks.
FYI. We found out and it was confirmed by the Khan lab who supplied the imaging data that the sample_id on the sample worksheet is the COG USI (Childrens Oncology Group) (Universal Identifier). Is there a way that we can label it as such or put a remark somewhere. This will help the CCDI folks to link to the images.
@ulrikew I just discussed this with @dclunie, and he suggests somewhere in the documentation describing the collection (ie, in our Zenodo record!) we mention that sample_id
attribute in the clinical spreadsheet data is the COG USI. Does this sound ok with you?
We do not want to change the spreadsheet, since it was used by David for conversion. And I do not want those transformations/relabeling to be done in George's scripts.
@G-White-ISB Mentioning it in the documentation works for me.
Ok, looking at the latest Excel file provided by @ulrikew, there are 13 different sheets. The clinical_measure_file ,methlation_array_file, publication, sequencing_file, study, study_admin, study_arm, and study_funding sheets all appear to contain headers but no data.
I assume the imaging file sheet is not useful. We are not providing the tiffs anyways.
@fedorov am I to write in that sample_id is the COG USI in the column description in the column_metadata table? We don't have much documentation outside the BQ tables themselves.
A column on the participant sheet is clearly mislabled. The header is ethnicity, values are 'Metastatic' or 'Non-metastatic'.
The spreadsheet was supplied by David Milewski david.milewski@nih.gov, so you might want to ask him about this (and cc myself and @ulrikew).
I wrote to him on 2023/02/27:
_"The ethnicity column in the Participant tag now contains the Metastasis_atdiagnosis value 😄
Also, I came across some cells in which there are new lines embedded in quoted text, which screws up simple line-oriented scripts that operate on extracted CSV files.
_If you get a chance, could you remove the new lines in the Samples tab for RMS2452, RMS2472 and others (with a second line that start with a parenthesis) in the sample_anatomicsite column cells please?"
but did not get a reply or updated form, so just worked around it.
He can also confirm for you that as he wrote by email 2023/08/11:
_"The COG USI is the same as the "sample.sampleid" in the table (example: "PAIZZZ"). This should enable us to link the sample to our Oncogenomics db."
If he is going to update the spreadsheet, he could add that information to the Dictionary tab entry for "sample_id" - not sure if you were planning on including the information in the dictionary in the BigQuery tables anyway
You might also want to ask him what the value of "study.phs_accession" should be, since I was told on a recent call that the "phs_accession" is what defines the scope of uniqueness of the "sample_id" (which is not globally unique).
If you don't hear back promptly, I would just go ahead and leave out the ethnicity column since it is clearly wrong (as well as being useless and peculiarly US-ethnocentric).
RMS clinical data was added for the v16 release
@G-White-ISB this query returns nothing - am I missing something?
SELECT * FROM `idc-dev-etl.idc_v16_clinical.column_metadata` where collection_id like "rms*"
The syntax is wrong. Change the '*' to a '%':
SELECT * FROM idc-dev-etl.idc_v16_clinical.column_metadata
where collection_id like "rms%"
Ah, of course!.. I was doing this while talking to Ulli, and didn't pay enough attention! Thank you!
Issues identified
@G-White-ISB it turns out the most valuable clinical data accompanying RMS-Mutation-Prediction is in the supplement 1 of this paper: https://aacrjournals.org/clincancerres/article/29/2/364/713962/Predicting-Molecular-Subtype-and-Survival-of. Let's plan to ingest that one in a future IDC release.
@ulrikew I am preparing an answer for Brian Furner, who asked how to get the COG USI identifiers, and I see that none of the columns for the clinical data accompanying RMS collection has descriptions of what the columns mean.
I think going forward it would be highly desirable for the submitters to be required to describe the semantics of the columns, which would then go into column_label
or something like that column. I understand we don't have those for legacy collections, but I think we should strive to improve for the collections we curate.
https://github.com/ImagingDataCommons/IDC-ProjectManagement/issues/1394#issuecomment-1674909405 Does this help? And yes, we need to require data dictionaries and an explanation of the used values.
Ulli, yes, I digged that out because I knew we talked about it. But from all I know, there is absolutely no chance anyone else finding that clinical data table will be able to figure this out.
Absolutely true!
see details in https://github.com/ImagingDataCommons/IDC-ProjectManagement/issues/1394