Display LC/MS Configurations for Lipidomics and Metabolomics #1525
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.
Fix #1514
Changes
Ingest
On ingest of
data_generation
records, there is now an additional check to see if slotshas_mass_spectrometry_configuration
andhas_chromatography_configuration
. In the case that one or both of those slots is populated, we'll do an additional lookup to theconfiguration_set
collection in mongo, and store theid
andname
of theconfiguration
document in theannotations
column of theomics_processing
table.They are stored in the
annotations
blob as:mass_spectrometry_configuration_name
mass_spectrometry_configuration_id
chromatography_configuration_id
chromatography_configuration_name
Storing these fields in the json blob has the advantage of not needing to update our database model. Our search already handles lookups into the
annotations
column (so we won't need to change how this is stored when it comes to implementing #1517), and theannotations
blob is available on the client and can be put into the UI as needed.Data Object Table (Vue component)
There is some additional processing to make the new data available to the vue component.