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

Add fastqc / sequali stats to fastq list row events for first 1 million reads #602

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

alexiswl
Copy link
Member

Write information to tables to prevent overfill of data in step functions object

…on reads

Write information to tables to prevent overfill of data in step functions object
@alexiswl alexiswl added bug Something isn't working feature New feature labels Oct 14, 2024
@alexiswl alexiswl self-assigned this Oct 14, 2024
@alexiswl
Copy link
Member Author

Ah @victorskl I don't think this can be cherry picked for O2, the underlying dynamodb uses the #589 syntax of the library orcabus id being the primary key for this particular database. So as you suggested, run 333 will need to be run once we're on O3

@victorskl
Copy link
Member

Yup. No blocker on Run 333 at ICA v2 side.
Happy to put into backlog and I will follow up once O3 concluded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants