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

Adding merge_sample() method #3274

Merged
merged 2 commits into from
Jul 10, 2023
Merged

Adding merge_sample() method #3274

merged 2 commits into from
Jul 10, 2023

Conversation

brimoor
Copy link
Contributor

@brimoor brimoor commented Jul 10, 2023

Resolves #3273

Adds a merge_sample() method for merging a single sample into a dataset via the same basic interface as merge_samples().

This method is useful for workflows that process one sample at a time, eg a cloud function that processes one sample at a time. It is not intended for use in a loop:

# Don't do this
for sample in samples:
   dataset.merge_sample(sample, ...)

# Use this instead
dataset.merge_samples(samples, ...)

@brimoor brimoor added bug Bug fixes feature Work on a feature request labels Jul 10, 2023
@brimoor brimoor requested a review from a team July 10, 2023 19:41
@brimoor brimoor self-assigned this Jul 10, 2023
@brimoor brimoor merged commit dfa4d96 into release/v0.21.3 Jul 10, 2023
@brimoor brimoor deleted the feature/merge-sample branch July 10, 2023 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug fixes feature Work on a feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant