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

MS-PINPOINT project added #76

Merged
merged 5 commits into from
Nov 25, 2024
Merged

MS-PINPOINT project added #76

merged 5 commits into from
Nov 25, 2024

Conversation

FordUCL
Copy link
Contributor

@FordUCL FordUCL commented Nov 20, 2024

Description

This is the requested Datacard for the MS-PINPOINT project operating at UCL. I am a PhD student working under Arman Eshaghi, who is the PI for the group.

Author Checklist

As author of this pull request I confirm:

  • This submission does not include, or link to any sensitive or person identifiable data
  • I have permission from my co-investigators on the project to publish this information
  • I understand that I have the right to add, amend or remove information published about the project

Reviewer Checklist

Reviewers should confirm the following:

  • These changes are limited to the creation of a new folder under /_projects/ and new content within that folder or these changes update one existing project
  • Running bundle exec jekyll serve on their local machine builds the site without any errors
  • The display of the new content appears normal and there are no obviously broken or missing elements on the page
  • The content has been read and checked for offensive material or spam

@docsteveharris
Copy link
Contributor

Thanks for this @FordUCL
Can I confirm that you built the data extracts using SQLSynthGen?

@stefpiatek stefpiatek self-requested a review November 21, 2024 09:21
Copy link
Contributor

@stefpiatek stefpiatek left a comment

Choose a reason for hiding this comment

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

Ah this looks great, thanks so much for getting this done. Couple of requests for formatting and tidying up, then made Steve's comment into a comment for resolving. Once all are resolved I'm happy to merge

_projects/MS_PINPOINT/Data/DistributionOfValues copy.csv Outdated Show resolved Hide resolved
_projects/MS_PINPOINT/info.md Outdated Show resolved Hide resolved
_projects/MS_PINPOINT/index.md Outdated Show resolved Hide resolved
_projects/index.md Outdated Show resolved Hide resolved
_projects/MS_PINPOINT/Data/FACT_RELATIONSHIP.csv Outdated Show resolved Hide resolved
@stefpiatek stefpiatek merged commit f0360e0 into SAFEHR-data:main Nov 25, 2024
1 check 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.

3 participants