Account for multiple sequencing runs in NEON surface water translator code #898
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The PR updates the code in the NEON surface water translator to account for creating
Manifest
/manifest_set
records for samples that have been sequenced multiple times. It also pulls NEON data from the latest/provisional release to update/create newDataGeneration
andDataObject
records for samples that were "ignored" in the first ingest.Details
...
Related issue(s)
Fixes microbiomedata/issues#1067
Related subsystem(s)
docs
directory)Testing
I tested these changes by...
Documentation
docs
directory)Maintainability
study_id: str
)# TODO
or# FIXME
black
to format all the Python files I created/modified