SynBioDex / pySBOL3

Native python implementation of SBOL 3.0 specification
MIT License
37 stars 16 forks source link

Object creation assumes that '/' implies a local ID segment. Do we want this? #290

Closed jakebeal closed 2 years ago

jakebeal commented 3 years ago

If we run sbol3.Sequence('foo/bar'), this will create an object with displayID bar, assuming that the foo part is the local component of the URI.

This is a reasonable behavior. However, another reasonable behavior would be to assume the person meant to make a displayID named foo/bar, which is not legal, and to complain.

I think that we want the more permissive (first) behavior, but we should consider it and document the choice one way or another.

tcmitchell commented 3 years ago

I agree that we want the more permissive behavior, which is the existing behavior. I also agree that this should be documented.