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

Default values need to be copied rather than linked #334

Closed jakebeal closed 3 years ago

jakebeal commented 3 years ago

From @jgladwig on #321

To point to a specific instance of the potential error... I see these two values in the ExperimentalRequest I pasted in my previous comment (under hasParameterValue):

<http://strateos.com/GrowthCurve/StringParameter9/StringValue10>,
<https://sd2e.org/ip3a4b5a12e48847e6ad834888245f4f74/MeasureValue10>,

From @jakebeal

http://strateos.com/GrowthCurve/StringParameter9/StringValue10 is indeed a potential problem.

What it looks like to me is that the default parameters are all getting referenced from the ExecutionRequest rather than copied into the ExecutionRequest. They should be copied over, making them child objects of the ExecutionRequest.

First thing to do is figure out if this is an OPIL issue or an Intent Parser issue.