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 run_meta as the dependency of PeaksPaired and store run_id into peaks_paired #63

Merged
merged 1 commit into from
May 2, 2024

Conversation

dachengx
Copy link
Collaborator

@dachengx dachengx commented May 2, 2024

close #36

The issue can be closed because we can use super runs to combine runs before bootstraping, so that the run boundary is not a problem.

@coveralls
Copy link

Coverage Status

coverage: 50.807% (-0.5%) from 51.284%
when pulling 2bd810e on run_meta_paired
into 198bcf1 on main.

1 similar comment
@coveralls
Copy link

Coverage Status

coverage: 50.807% (-0.5%) from 51.284%
when pulling 2bd810e on run_meta_paired
into 198bcf1 on main.

@dachengx dachengx merged commit 4394f39 into main May 2, 2024
7 checks passed
@dachengx dachengx deleted the run_meta_paired branch May 2, 2024 12:34
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.

Make sure the bootstrap AC will not extend outside original runs
2 participants