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.

Sign up for GitHub

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

Jump to bottom

Feat (metadata-manager): Update model #535

Merged
merged 14 commits into from
Sep 20, 2024
Merged

Feat (metadata-manager): Update model #535

merged 14 commits into from
Sep 20, 2024
+1,205 −725

Conversation

williamputraintan
Copy link
Member

@williamputraintan williamputraintan commented Sep 9, 2024 •
edited
Loading

Update the model to support more data ingestion from the tracking sheet.

Sheet Header Table Field Name
SubjectID Individual individual_id
ExternalSubjectID Subject subject_id
SampleID Sample sample_id
ExternalSampleID Sample external_sample_id
Source Sample source
LibraryID Library library_id
Phenotype Library phenotype
Workflow Library workflow
Quality Library quality
Type Library type
Coverage (X) Library coverage
Assay Library assay
ProjectName Project project_id
ProjectOwner Contact contact_id

@williamputraintan williamputraintan added feature New feature metadata_manager anything related to metadata manager labels Sep 9, 2024
@williamputraintan williamputraintan self-assigned this Sep 9, 2024
@williamputraintan williamputraintan marked this pull request as ready for review September 9, 2024 00:06
@williamputraintan williamputraintan marked this pull request as draft September 9, 2024 07:37
@williamputraintan williamputraintan linked an issue Sep 12, 2024 that may be closed by this pull request
Metadata Manager Model Refactor #541
Closed
@williamputraintan williamputraintan marked this pull request as ready for review September 19, 2024 00:15
reisingerf
reisingerf approved these changes Sep 19, 2024
View reviewed changes
Copy link
Member

@reisingerf reisingerf left a comment

Choose a reason for hiding this comment

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

Let's go!

@alexiswl
Copy link
Member

LGTM!

victorskl
victorskl approved these changes Sep 19, 2024
View reviewed changes
Copy link
Member

@victorskl victorskl left a comment

Choose a reason for hiding this comment

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

Alright. Let's do this!

@williamputraintan williamputraintan merged commit 0af058d into main Sep 20, 2024
6 checks passed
@williamputraintan williamputraintan deleted the feat/model-changes branch September 20, 2024 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@victorskl victorskl victorskl approved these changes

@reisingerf reisingerf reisingerf approved these changes

Assignees

@williamputraintan williamputraintan

Labels
feature New feature metadata_manager anything related to metadata manager
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Metadata Manager Model Refactor
4 participants
@williamputraintan @alexiswl @victorskl @reisingerf