vrs icon indicating copy to clipboard operation
vrs copied to clipboard

Create schema-level constraint for start/stop on SequenceLocation within Adjacencies

Open Mrinal-Thomas-Epic opened this issue 11 months ago • 2 comments

Currently, the SequenceLocation within an Adjacency is expected to have either start or stop populated, but not both. The constraint that both should not be populated is not specified anywhere in the schema, which could lead to confusion (e.g., discussion #471).

Mrinal-Thomas-Epic avatar Mar 05 '24 15:03 Mrinal-Thomas-Epic

@Mrinal-Thomas-Epic this is great, thanks. Would you be interested in drafting the schema change?

ahwagner avatar Mar 05 '24 19:03 ahwagner

I'll take a stab at draft the schema change. Before I write it out, do you or @d-cameron have any thoughts on whether it should just be implemented as just a constraint on the use of SequenceLocation within Adjacency, or whether we should make a different class for a point + direction on a sequence?

Mrinal-Thomas-Epic avatar Mar 25 '24 17:03 Mrinal-Thomas-Epic

@Mrinal-Thomas-Epic I think a constraint definition makes the most sense for forward compatibility.

ahwagner avatar Aug 22 '24 19:08 ahwagner