Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linked cohorts showcase #27

Merged
merged 13 commits into from
Nov 10, 2023
Merged

Conversation

z-w123
Copy link

@z-w123 z-w123 commented Oct 30, 2023

Hi,
this PR is for the sandbox version of the linked cohort data showcase page.

thanks in advance,
Zahra

@z-w123 z-w123 requested a review from bedroesb as a code owner October 30, 2023 21:46
description: Here we describe the first example of linking and sharing a cohort data set via the EMBL-EBI infrastructure.
affiliations: [ELIXIR Europe]
description: Here we describe the first example of linking and sharing a COVID-19, multi-omics cohort data set via the EMBL-EBI infrastructure.
affiliations: [EMBL-EBI]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't forget to add information about this affiliation to the _data/affiliations.yml file!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah thanks, done!

@@ -1,8 +1,8 @@
---
title: Linked Cohort Data
title: Linked pathogen and host data across archives - a multi-omics, SARS-CoV-2 cohort case study
contributors: [Gabriele Rinck, Zahra Waheed]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same goes for these contributors in the _data/CONTRIBUTORS.yml file :)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done too :)

@bedroesb
Copy link
Member

bedroesb commented Nov 6, 2023

@z-w123 thanks for this! Hopefully we can soon create a PR from you branch to the original repo after we approve the try-out on this repo! See my small comments to finalize

@z-w123
Copy link
Author

z-w123 commented Nov 8, 2023

Thanks @bedroesb! Have addressed the comments

@bedroesb
Copy link
Member

bedroesb commented Nov 8, 2023

lets have a look how they look like live!

@bedroesb bedroesb merged commit 7df89d3 into ELIXIR-Belgium:main Nov 10, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants