Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend Sequence Location documentation to discuss optional sequence reference #564

Open
ahwagner opened this issue Oct 16, 2024 · 0 comments
Assignees
Labels
Product Review Features requested as part of product review
Milestone

Comments

@ahwagner
Copy link
Member

SequenceLocation should require either sequenceReference or sequence to be populated

No. Sequence is just descriptive of the content described by the SequenceLocation, it is not definitive like a sequenceReference is. SequenceReference is optional due to its use in CisPhasedBlock Alleles. We recognize that the documentation needs to be extended to discuss this nuance and we are taking that issue on in #564.

Originally posted by @ahwagner in #533 (reply in thread)

@ahwagner ahwagner added the 2.0 proposed changes and feature requests for v2 label Oct 16, 2024
@ahwagner ahwagner self-assigned this Oct 16, 2024
@ahwagner ahwagner added this to the 2.0 milestone Oct 22, 2024
@ahwagner ahwagner added the Product Review Features requested as part of product review label Oct 22, 2024
@ahwagner ahwagner added this to VRS Oct 22, 2024
@ahwagner ahwagner removed the 2.0 proposed changes and feature requests for v2 label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Product Review Features requested as part of product review
Projects
Status: TODO
Development

No branches or pull requests

1 participant