Closed turbomam closed 4 years ago
I'm thinking now that I commented drivetrain:hasRequiredInput drivetrain:MulScoMul
because it caused a performance degradation.
And the replacement of hard-coded TURBO column terms with drivetrain:ClassResourceList
really was complete. The issue I posted about that was just a manic oversight.
In the spirit of #67
Sorry for lumping several things together here.
Requests
acornOntology.ttl
anymore, but they are still mentioned in Drivetrain error messages.https://github.com/PennTURBO/Drivetrain/HealthcareEncounterRegDen is not present as an input and has a range list size that is not 1
Incomplete/potentially conflictual?
See
MAM 20191120
comments inpost_icbo_synthea_omop_cnp_transformation_instructions.ttl
andturbo_valid_graph_specification.ttl
condition_occurrence_id
... variable will be an integer, not a string. Didn't we talk about asserting a general type in the valid graph specification and a more specific type in the transformation instructions?The object of connection https://github.com/PennTURBO/Drivetrain/DiagKeySymbHasRepresentation is not a literal, but the connection is a datatype connection.
do you remember whydrivetrain:hasRequiredInput drivetrain:MulScoMul
is commented out? Did I do that, or you? I feel like we discussed something like that in the past but can't remember any more than that.I thought I replaced hard-coded TURBO column terms in the valid graph specification withdrivetrain:ClassResourceList
s, but I still see the hard-coded values, just commented out, as indrivetrain:RxAboutnessAxiomDbXrefPartOfColumn
. If you have any ideas for finishing this as a bulk operation, please let me know.Hopefully these are harmless:
# baseURI
and@prefix :
acornOntology.ttl