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

U19 ETL #76

Merged
merged 74 commits into from
Aug 29, 2024
Merged

U19 ETL #76

merged 74 commits into from
Aug 29, 2024

Conversation

Sun-flow
Copy link
Contributor

closes #42

Converting the U19 ingest scripts into a re-usable ETL module.

@Sun-flow Sun-flow self-assigned this Jun 21, 2024
@Sun-flow Sun-flow requested a review from jtyoung84 August 15, 2024 23:02
Copy link
Contributor

@mekhlakapoor mekhlakapoor left a comment

Choose a reason for hiding this comment

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

I was able to test it with real data and it worked! We'll need to discuss whether to include this in the GatherMetadataJob in future

@mekhlakapoor mekhlakapoor dismissed jtyoung84’s stale review August 29, 2024 00:24

Added U19 to [all]. Discussed that everything else looks ready to merge

@mekhlakapoor mekhlakapoor merged commit 6823113 into dev Aug 29, 2024
3 checks passed
@mekhlakapoor mekhlakapoor deleted the feat-42-U19-ETL branch August 29, 2024 00:25
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.

Write ETL for U19 Smartspim data
3 participants