search

SynBioDex-archive / libSBOLj

deprecated libSBOL Java library
http://SynBioDex.github.com/libSBOLj
2 stars 0 forks source link

SequenceAnnotation.setFeature should be removed #4

Closed mgaldzic closed 13 years ago

mgaldzic commented 13 years ago

SequenceAnnotation.addFeature exists, features should be added one at a time. Setting all of them at once is undersirable.

mgaldzic commented 13 years ago

When reading GenBank, there will only be one feature per Annotation. If we have another Use case that necessitates one or the other option, that would help convince me.

mgaldzic commented 13 years ago

If we change the model to SA-SF as 1to1 then its ok. * Cesar advocated for this to be 1to1, as no use case is currently there for absolutely overlapping features.

mgaldzic commented 13 years ago

I have removed the .setFeature method as for now it would be confusing to have two methods for creating the SA-SF link