SD2E / experimental-intent-parser

A tool that combines a word-processing interface with structured tables and assisted linking to definitions to provide a simple interface for incremental codification of experiment designs.
BSD 3-Clause "New" or "Revised" License
4 stars 0 forks source link

IP cannot parse "M9 Minimal Media w 50mg per L TRP" #302

Closed mwes closed 3 years ago

mwes commented 3 years ago

We are running into an issue trying to submit an ER for execution at Strateos:

https://docs.google.com/document/d/1_c1A9F9PBSh6vc-mDfnMcJxlurqtXMANQQ9aFJOTcUk

What we're seeing: When the control table has a content element linked inside it: "M9 Minimal Media w 50mg per L TRP" that links to https://hub.sd2e.org/user/sd2e/design/Modified0x20M90x20Media0x200x2B0x2050mg0x2FL0x20Trp/1 the IP returns an error:

{"errors": ["Controls table has invalid Contents value: Unable to parse M9 Minimal Media w 50mg per L TRP\n"]}

From what I can see, this should not return an error - this looks like a viable contents entry to me. I'm not sure where that newline is coming from - the entry has no such newline.

I've verified the that SD2 program dictionary does indeed contain an entry for "M9 Minimal Media w 50mg per L TRP" that maps to the above URI. The SD2 Common Name is the same for this value.

mwes commented 3 years ago

Edit: we may be able to get around this by using a different set of values (discussing this now with Josh and Dan), but I suspect this issue will still remain and could affect us in the future. Something is potentially confusing the contents parsing here.

mwes commented 3 years ago

Edit 2: We did indeed work around for now this by using an alternative value ("modified_m9_media_with_50mg_per_l_trp"). I'm not sure why the original value does not work.