Cleanup: EMSL identifiers #2299
bmeluch
started this conversation in
Schema Best Practices
Replies: 1 comment
-
Hey @bmeluch I have one thought right now. You mentioned in the first part "We would rather have no external identifiers for users than confusing ones". I am not sure what all is meant by confusing ones, but I am thinking from the view of what we are doing in the Material Processing Squad that we should still keep alternative identifiers even if we can't make them resolvable. Some of them still mean something to us, as developers in EMSL, and we are using them/may use them in our squad to help track down that data/information to help us find more context around it. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Skip to the bottom for actual discussion questions
this first part is just my notes
Making sure I understand the current situation re: EMSL identifiers in NMDC
EMSL prefixes defined in the schema
First two resolve, last two do not.
EMSL-specific identifier slots in the schema
emsl_biosample_identifiers
- regex pattern restriction does not specify prefixemsl_project_identifiers
- regex pattern restriction requiresemsl.project:
, does not seem to be populatedExample EMSL identifiers currently in Mongo
Biosample
Does not resolve.
DataGeneration
Does not resolve. But it is easy to derive a working MyEMSL link from that six-digit number (DMS dataset ID) and convert this to a
my_emsl:
formatted identifier if that would be helpful.DataObject
First resolves, second does not.
Questions
output
inemsl:output_749047
, but our processed DataObjects don't correspond to anything stored at EMSL afaik. So all this does is say what raw data this processed data came from. Can we remove these and instead use thehas_input/output
connections through WorkflowExecution?Possible actions
Some combination of the following?
emsl:
andemsl_uuid_like:
prefixes and any slot values that use them.@lamccue @mslarae13 @brynnz22 thoughts?
Beta Was this translation helpful? Give feedback.
All reactions