ebi-ait / hca-ebi-wrangler-central

This repo is for tracking work related to wrangling datasets for the HCA, associated tasks and for maintaining related documentation.
https://ebi-ait.github.io/hca-ebi-wrangler-central/
Apache License 2.0
7 stars 2 forks source link

GSE138262 - WehrensHypertrophicCELSeq2 #1197

Closed arschat closed 9 months ago

arschat commented 11 months ago

Project short name:

WehrensHypertrophicCELSeq2 (VanRooij-Human-CELseq2)

Primary Wrangler:

Arsenios

Secondary Wrangler:

Wei

Associated files

Published study links

Key Events

arschat commented 11 months ago

Publication uses data from other publications

but neither sequence files and analysis files do not include this information.

I will not consider the healthy data, and just wrangle the 5 Hypertrophic cardiomyopathy donors.

arschat commented 11 months ago

Moved files to hca-util upload area and removed from hca-ncbi-cloud-data bucket. hca-util-upload-area uuid: c7512d6d-0c1f-4f63-b664-eb43a6205014

arschat commented 11 months ago

Graph valid & ready for sec review

Wkt8 commented 11 months ago

Assigning self for secondary review

Wkt8 commented 11 months ago

Hey Arsenios! Looks really good, agree with your choice for only wrangling the HCM donors. The paper laid out a lot of information and I think you modelled it really well.

The only points I have are on the analysis_protocol tab and one point on the project tab:

  1. The analysis protocol for barcode_matrix_generation isn't being used in any of the processes.
  2. The description for the analysis protocol 'transcript_matrix_generation' feels like it could be fleshed out a bit more, but I'll leave it to your discretion.
  3. One point on the project tab: leaving 'the Netherlands' with the 'the' lower-case stylistically doesn't feel right to me. Would change it to 'The Netherlands'
arschat commented 11 months ago

Hi Wei, thanks for the comments! Fix those, exported, and import form sent!

arschat commented 9 months ago

Verified in browser